Trigger load event after checking the wallet address

[raducristianpopa]

At the moment, the load event is triggered automatically when the WM polyfill is injected which is not the intended behaviour. The monetization link should be checked before triggering the load event - spec.

• Inject polyfill on document_start (before window.onload) (--- from Allow checking for WM support before window's load event #78 , implemented here)

[tselit]

In progress: @dianafulga will discuss with Radu (25 Jan) before in order to update the estimated effort.

[dianafulga]

Missing a clear definition of the "verification of the monetization link". Currently the verification code is spread across multiple files and moments in the user flow like so:

• if WM is disabled in the extension (not yet implemented - TBD in the new flow), time: to be decided, location: to be decided (if WM disabled, don't proceed to other checks)
• if valid format for a link (ex: having http/https protocol), time: when opening a new page, location: TabManager
• if link tag in page, time: when opening a new page, location: background
• establish session (which qualifies as verification according to the specs), time: when user clicks on connect (to be changed with the new flow), location: background

According to the specs, if the verification fails, we should emit an error event, but currently is not implemented at every step during all the steps mentioned above, to be decided if and where we want to implement it

[dianafulga]

The ticket will considered solved with the current PR as the load event has been moved to be triggered after a session has been established.