Kill and restart the codehint worker thread every 30 hint requests.

This avoids some memory leaks that we were encountering.  Still looking into those, but this should at least stabilize things and prevent the memory from growing so much it crashes brackets.

To avoid a noticeable performance hit when we restart the worker, we instead start a new worker, and only switch to it after it has finished intializing (including priming the pump).  So when we switch between the old and new worker it should be unoticeable to the user.

Had to move around a lot of code in ScopeManager - there are not a lot of actual changes, but the diff is messy due to this.
Code review cleanup - delete commented out line
Merge pull request #4025 from eztierney/restart-worker

[js code hints] Kill and restart the codehint worker thread every 30 hint requests.