1541748 - New tab and restored tab notified via tabs.onCreated can have invalid (too large) index

Status: VERIFIED FIXED

(WebExtensions :: Compatibility, defect, P3)

Description

[YUKI "Piro" Hiroshi]

Attachment: test.zip: testcase addon

Firefox notifies new tab or restored tab to listeners of tabs.onCreated, but the notified tab can have invalid index which is different from actual index on the tab bar.

Steps to reproduce

1. Register an listener to tabs.onCreated, like:
   chrome.tabs.onCreated.addListener(tab => console.log(tab))
2. Open new tab with too larget index, like:
   chrome.tabs.create({ index: 9999999 })
3. Open four ore more tabs.
4. Close all tabs opened at the step 3, with the reversed order they were opened.
   (In other words, close tabs from the last, to the first.)
5. Open the history menu and restore the initially closed and last opened tab.

Actual result

Both tabs have their index larger than tabs.length - 1.
The tab opened at the step 2 has 9999999.
The tab restored at the step 5 has the index same to the one when it was closed.

Expected result

Both tabs have their index same to tabs.length - 1.

Environment

• Nightly 68.0a1 on Ubuntu 16.04LTS
• Firefox 66.0.2 on Ubuntu 16.04LTS

Additional information

Google Chrome (Chromium) works as expected. I've attached a testcase extension, and I got a console output like:

opened with index 2
there are 3 tabs
opened with index 2
there are 3 tabs

On the other hand, Firefox reports:

opened with index 9999999 <= larger than tabs.length-1
there are 3 tabs
opened with index 4 <= larger than tabs.length-1
there are 3 tabs

Comment 1

[Fanolian]

Is it a dupe of bug 1504775?

Comment 2

[YUKI "Piro" Hiroshi]

The bug 1504775 looked to cause this. But the patch contains changes only on the XUL layer, and there is no guarantee about compatibility with Chromeium's extensions API. One more automated test on the WebExtensions API layer looks better for more clear compatibility.

Comment 3

[Shane Caraveo (:mixedpuppy)]

Are you saying that this behavior started after the patch from bug 1504775? I don't see how that is logically possible.

Comment 4

[YUKI "Piro" Hiroshi]

(In reply to Shane Caraveo (:mixedpuppy) from comment #3)

Are you saying that this behavior started after the patch from bug 1504775?

No, I meant its opposite. I think that the patch coincidentally fixes this WebExtensions API compatibility bug, and a new test to verify the compatibility is required. Thus I'm trying to write a patch for future consistency. Is such a "only including test" patch acceptable?

Comment 5

[YUKI "Piro" Hiroshi]

Attachment: Bug 1541748 - Add tests to verify WebExtensions API compatibility around new tab index

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fd7d4206ad59
Add tests to verify WebExtensions API compatibility around new tab index r=mixedpuppy

Comment 7

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/fd7d4206ad59

Comment 8

[Madalin Cotetiu]

This issue is verified as fixed on Firefox 68.0a1 (20190508111321) under Win 7 64-bit and Mac OS X 10.14.1
I have used the test add-on from the attachment and the issue is no longer reproducing. The following is displayed in the console:

opened with index 2
there are 3 tabs