CB-14190: (windows) Fix bug where double clicking <select> exits app

[iainconnor]

Double clicking on <select> elements will no longer exit the application.

Platforms affected

• Windows.

What does this PR do?

Previously, there was code which I assume was meant to detect a click outside the webview (and on the browser wrapper), and close the webview if such a click occurs. This makes sense if the webview is treated like a popup, but the code did not check if the target of the click was the webview itself. For whatever reason, double clicking a <select> element triggered this behaviour, incorrectly closing the webview when the user is trying to interact with it.

What testing has been done on this change?

• Tested that double clicking a <select> element no longer triggers the exit event.
• Tested that a non-fullscreen inappbrowser still triggers the exit event if clicked outside of.

Checklist

• Reported an issue in the JIRA database
• Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
• Added automated test coverage as appropriate for this change.

[janpio]

Hey, I just fixed the problem that caused Android tests to fail in master. Could you rebase this PR please? This should get rid of the Android failures and possibly fix all test failures for this PR.

[janpio]

Closing and re-opening to trigger a new CI/test run with new PR merge.