Chrome: Deprecation - Unload event listeners are deprecated and will be removed about applicationinsights-js

[MSNev]

Hello, as a generic library we will not be removing the backward compatibility support for "hooking" the unload event.

However,
we have a really badly documented (like barely) configuration option that we introduced in v3.0.3 which allows you to tell use which events you "don't" want us to hook. So you can avoid the "warning" by simply adding the configuration option

ApplicationInsights-JS/shared/AppInsightsCore/src/JavaScriptSDK.Interfaces/IConfiguration.ts

Lines 156 to 161 in 7f804d8

	/**
	* [Optional] An array of the page unload events that you would like to be ignored, special note there must be at least one valid unload
	* event hooked, if you list all or the runtime environment only supports a listed "disabled" event it will still be hooked, if required by the SDK.
	* Unload events include "beforeunload", "unload", "visibilitychange" (with 'hidden' state) and "pagehide"
	*/
	disablePageUnloadEvents?: string[];

disablePageUnloadEvents: [ "unload" ],

And once the event is no longer present (which we will still attempt to hook it), it just won't be hooked.

I'm going to mark this issue as a documentation issue as we REALLY need to document this better.
Some readme level documentation exists for the @microsoft/1ds-core-js package which is now co-located in this repo and this is the same configuraiton option as the 1ds-core-js is built on-top of applicationinsights-core-js
https://github.com/microsoft/ApplicationInsights-JS/tree/main/shared/1ds-core-js#iextendedconfiguration