WM ext browser support - Edge

[tselit]

Context

WM extension compatibility with the Microsoft Edge browser.

Todos

TBC

[tselit]

Bibi to do: test on Edge and then should be ready to close

[sidvishnoi]

To test: How it works in split-window mode (https://github.com/interledger/web-monetization-extension/pull/366#discussion_r1673836423)

[raducristianpopa]

In split view, the extension is only taking into the consideration only the first pane from the split view when updating icon. If we are on a non-monetized website in the first pane and on a web monetized one in the second one, the icon will have the non-monetized website identifier.

[raducristianpopa]

Arc's split view should be checked as well. Vivaldi's tab tiling as well.

[RabebOthmani]

Blocked as we wait for the MS support team

[sidvishnoi]

https://github.com/microsoft/MicrosoftEdge-Extensions/discussions/102 🤷🏽‍♂️🤦🏽 https://github.com/uBlockOrigin/uBlock-issues/issues/2945

[sidvishnoi]

Found the sender port in Runtime.Port message handler returns correct tab ID 🎉

port.onMessage.addListener((message, sendingPort) => {
   console.log(sendingPort.sender?.tab?.id);
});

So, we can make a short connection on content script inject and store tabId for use in future messages?

Edit: or instead of tabActivated event, rely on focusChange messages from content script.

[sidvishnoi]

Everything works once we get right tabId, except Edge doesn't update the icon (even with correct tabId!)