Add ChromeWidgetMessageInterceptor to WinForms Example

[jankurianski]

Addresses #570

[jankurianski]

Anyone have corrections or thoughts before I merge this?

[amaitland]

Only question I have is would it be better to explicitly release the handle?

[amaitland]

If it's working then I'm happy to merge is as, few formatting things, they can be sorted later 👍

[jankurianski]

Only question I have is would it be better to explicitly release the handle?

It is working without releasing the handle, but it may be inadvertently preventing garbage collection. So I have made it explicitly release the handle now.

[jankurianski]

Add browser.HandleDestroyed -= BrowserHandleDestroyed;?

Good catch 👍

[amaitland]

Might be worth adding the extra example from #1098 (comment)

Can be commented out, just as a user reference?

[jankurianski]

Might be worth adding the extra example from #1098 (comment)

Can be commented out, just as a user reference?

Good thinking. I might actually change the ChromeWidgetMessageInterceptor to accept an Action<Message> delegate, then it can be up to the caller to do what they wish with the widget messages.

[jankurianski]

I added a comment about mouse events.

I also moved moved the ToolStrip-closing logic into BrowserTabUserControl. Any thoughts?

[amaitland]

Looks fine 👍