Addon broken in Waterfox Current (2019.10)

[Beck-919]

• Temporary Containers Version: 1.7
• Firefox Version: Waterfox Alpha 2019.10

Actual behavior

New tabs not opening in new containers (Automatically open is set)

Expected behavior

New tabs open in new containers

Steps to reproduce

1. Open new tab after ticking Automatic Mode in addon settings page
2. New tab opens normally, does not open in container

Notes

1st issue ever raised on github, please ask if any more details required.
Issue not reproducing on normal Firefox

[stoically]

Could you check if you have the same issue as #332?

[Beck-919]

New tabs are set to Waterfox Home (Default).

I have also put up a post on the r/waterfox, so I will check from that side also

[stoically]

So I've checked and the reason is that Waterfox doesn't support browser.contextMenus.onShown, because it relies on a hopelessly outdated version of Firefox. I'm actually a bit reluctant to support that, but I'll check if its enough to make an exception-check for the presence of contextMenus.onShown.

[Beck-919]

Waterfox Current (gave wrong name as Alpha) 2019.10 is based on Firefox 68.0b1, that too doesn't support it?

[stoically]

Ah, didn't realize there's an updated version. TC requires Firefox 67, so it should work fine. The reason it doesn't work is the fix that was made for Icecat in #307. Waterfox returns { name: "Waterfox", vendor: "Mozilla", version: "68.0", .. } on the browser.runtime.getBrowserInfo() API, but the container ids are still prefix with firefox- instead of waterfox-. I'd say that should be fixed in Waterfox, but I look into adding an exception in TC to make it work.

[Beck-919]

Ok, thanks. Please leave a comment once you have fixed it. I will update and return to you then.

[stoically]

FWIW, a workaround to get it to work now for a running Waterfox until the browser is restarted, is executing tmp.containerPrefix = 'firefox'; on the debug console.

[Beck-919]

Ok, I will do that for now. But please do fix it.

[stoically]

Just realized, there's a workaround that even survives browser restarts:

tmp.storage.local.containerPrefix = 'firefox';
tmp.storage.persist();

on the debug console should do the trick.

[Beck-919]

Sure, thanks. But it won't matter once the fix is released right? Or will I have to revert it back?

[stoically]

Glad to help. A possible fix would do the same, so you're good.

[Beck-919]

Worked like a charm. Will wait for the fix. For now all is well :)