Temporary-containers: Focus/Cursor not working in Firefox 67 beta

[BryanQuigley]

• Temporary Containers Version: 0.91
• Firefox Version: 67.0b3-1 (snap 193)
• Operating System + Version: Ubuntu 19.04 Dev
• Non-default Options/Preferences in Temporary Containers: Automatic on.

Expected behavior

In Firefox 66:

1. Install https://addons.mozilla.org/en-US/firefox/addon/temporary-containers/
2. Enable Automatic mode
3. Ctrl - T for new tab, note how you immediately have focus/cursor in the new tab and can type.

Actual behavior

Upgrade to Firefox 67 beta
..

1. Ctrl - T for new tab, note how you can't immediately type and have to click on the new tab.

Notes

I previously had a similar (but worse) issue tracked in Bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1489876
That was resolved, and so far this one only seems to impact Temporary Containers add-on. Never did figure out what fixed it so could be related.

[stoically]

Did you try with Multi-Account Containers and opening a new container tab from their popup? TC just uses the tabs.create API for WebExtensions and sets the active parameter - there's no API to focus the omnibox. I'm afraid this can't be fixed in an Add-on.

[BryanQuigley]

Yup, Multi-Account Containers still works.

I'm happy to report to bugzilla (is that what makes sense?) but this time I can only reproduce it against this one add-on.

[stoically]

Did some debugging and it turns out that in Firefox Beta 67.0b3 and Firefox Nightly 68.0a1 (2019-03-22) the tabs.onCreated event fires with the active property set to false for Ctrl+T and clicking the new tab icon. TC relies on the correct active value being set, since it hands it down to tabs.create. I'd consider this a serious regression and would advise reporting it to Bugzilla.

[BryanQuigley]

Thanks for the debugging analysis. Reported https://bugzilla.mozilla.org/show_bug.cgi?id=1538378

[stoically]

Thanks for reporting. Maybe consider changing the bug title to something more accurate like tabs.onCreated fires with active set to false for new tabs.

[BryanQuigley]

updated with recommendation

[stoically]

Closing since it's not a TC issue.