Remove asynchronous search in Dynamo

### Changes

This pull request removes the `DispatcherTimer` in `SearchView` and the asynchronous `SearchAndUpdateResults` method in `SearchViewModel`.   Instead, all search is done synchronously.

### Results

I found search to be feel far more responsive, while also fixing issue #1092.  Notably, this removes any need to create a search Task or lock any data structures in the SearchViewModel class.  

However, it's possible we could run into performance issues on slower machines.  Is there a way to test this?  

@Benglin @ikeough
A minor thing... I was imagining this to be a ```while (catName.Length > 50)``` just in case we got a really nested class.