Remove update check from about box.

The about box should not conduct its own update check. Doing so creates
the possibility of two simultaneous updates occurring, and the update
download file locking.

This commit also removes two behaviors, ESC to close, clicking on
version number to install update,  which were not thread-safe.
Merge pull request #2350 from DynamoDS/MAGN-4518

Remove update check from about box.
Remove logger reference from constructor.

As suggested in #2350.
Fix one UpdateManager test.

Test was broken due to the patch that was added to fix the update
download race condition.
Cleanup unused call to check for update.
Fix MAGN-4522 When building Dynamo using visual studio, DynamoSandbox does not open, if there is no folder Autodesk in programfiles
Merge branch 'master' into Revit2015