Started the self-serve run with these changes

@reddyashish - actually I have been thinking about this a bit more:

This seems weirdly specific for the WorkspaceReferences Extension to have code that lives in homeworkspaceModel and DynamoView...

Instead, what about just adding some code to the WorkspaceReferences Extension to watch for the close or clear events on the current homework space and close when that occur, I think that is a better design if this functionality is only being implemented for this extension thats where I would look for it - on the other hand if this is a general implementation for all extensions then it belongs where you put it.

I agree in general that this behavior is actually not desirable for all extensions, so my vote would be to implement this as part of the extension logic, and not as part of DynamoCore.

(if you have to add new events to homeWorkspace - like Cleared - then do that.)

@mjkkirschner made the necessary code changes. Running the self-serve now and will add some tests to this.

I have some code changes touching same files in #10279. It maybe good for us to merge in turns to avoid conflicts

@reddyashish I see @QilongTang renamed the method already - LGTM - lets just be sure theres no conflicts.

@reddyashish Sorry for the conflicts! Once that is resolved, you should be good to merge

@QilongTang @mjkkirschner No worries, thanks for the changes. I will pull in your changes.

Closing this and opening a new one to resolve the conflicts. #10280