Not register event handler for Dispatcher.ShutdownStarted event

It is part of effort to fix [memory leak issue on Revit](http://adsk-oss.myjetbrains.com/youtrack/issue/MAGN-3404)...``RevitDynamoModel`` is referenced by many objects, when looking at the reference retention path, ``Dispatcher`` is one if them. 

Related pull requests:
  - [Inherit from NotificationObject to avoid memory leak in data binding] (https://github.com/DynamoDS/Dynamo/pull/2890)
  - [Clear task queue when Scheduler is shutdown] (https://github.com/DynamoDS/Dynamo/pull/2881)

As ``Dispatcher`` is associated with thread, it is sort of static object, registering event handler for ``Dispatcher.ShutdownStarted`` prohibits the view from being garbage collected even it is unloaded. Plus, ``Dispatcher.ShutdownStarted`` is only fired when Revit application is shut down, it doesn't make sense to do "unload" things in it. Replace them with ``Unloaded`` event.

Reviewers:

- [ ] @ikeough 
- [ ] @Benglin 
I would have hoped this ```dynNoteView_Unloaded``` gets named ```OnNoteViewUnloaded```, and ```dynNoteView_Loaded``` be named ```OnNoteViewLoaded```. But then I think you didn't do that so as to keep the naming convention in the class, which if fair enough (I'd also move the *Unloaded* to after *Loaded*).