Call disconnect on previous handler with Shell

[PureWeen]

Description of Change

When swapping in a new handler for shell call disconnect on the previous handler so that the xplat events don't trigger operations on the old set of handlers. There's a larger issue here where I think we need to call SetVirtualView(null) on the previous handler. Something not quite as involved as calling "Disconnect".
#8456

Testing notes

To test this

• fire up a shell app
• go into developer settings on android and select the option to always destroy activities

Issues Fixed

Fixes #7940

[jsuarezruiz]

Description of Change

When swapping in a new handler for shell call disconnect on the previous handler so that the xplat events don't trigger operations on the old set of handlers. There's a larger issue here where I think we need to call SetVirtualView(null) on the previous handler. Something not quite as involved as calling "Disconnect". #8456

Testing notes

To test this

• fire up a shell app
• go into developer settings on android and select the option to always destroy activities

Issues Fixed

Fixes #7940

I think I have found something similar with the WebView on Windows. Get a crash closing a Window with a WebView, mostly because doing that we are not invoking the Disconnect method. Will send a PR with changes but definitely I think this apply to the rest.

[jsuarezruiz]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[rmarinho]

D:\a\_work\1\s\src\Controls\tests\DeviceTests\HandlerTestBase.cs(205,12): error CS1501: No overload for method 'SetupWindowForTests' takes 3 arguments [D:\a\_work\1\s\src\Controls\tests\DeviceTests\Controls.DeviceTests.csproj]
D:\a\_work\1\s\src\Controls\tests\DeviceTests\HandlerTestBase.cs(205,12): error CS1501: No overload for method 'SetupWindowForTests' takes 3 arguments [D:\a\_work\1\s\src\Controls\tests\DeviceTests\Controls.DeviceTests.csproj]
D:\a\_work\1\s\src\Controls\tests\DeviceTests\HandlerTestBase.cs(205,12): error CS1501: No overload for method 'SetupWindowForTests' takes 3 arguments [D:\a\_work\1\s\src\Controls\tests\DeviceTests\Controls.DeviceTests.csproj]

[PureWeen]

@rmarinho fixed!

[jzhao8402]

When the fix will be available?