1272873 - sdk/tabs not fire load event

Status: RESOLVED INCOMPLETE

(Add-on SDK Graveyard :: General, defect, P2)

Description

[ettoolong]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:46.0) Gecko/20100101 Firefox/46.0
Build ID: 20160502172042

Steps to reproduce:

my add-on script:

let self = require("sdk/self");
let tabs = require("sdk/tabs");
let winUtils = require("sdk/window/utils");
let tabUtils = require("sdk/tabs/utils");
let { modelFor } = require("sdk/model/core");

onTabOpen = function(tab) {
    console.log("onTabOpen tabId = " + tab.id);

    tab.on("load", function(tab){
      console.log("tab onload");
    });
};

tabs.on("open", function (tab) {
  onTabOpen( tab );
});

exports.main = function (options, callbacks) {
  //find all onened tab that before add-on install,
  //call onTabOpen to register load event for these tabs
  let allWindows = winUtils.windows(null, {includePrivate:true});
  for (let chromeWindow of allWindows) {
    let openedTabs = tabUtils.getTabs( chromeWindow );
    for(let i=0; i < openedTabs.length; ++i) {
      onTabOpen( modelFor(openedTabs[i]) );
    }
  }
}
//
step 1, open firefox, console log show "onTabOpen tabId = -3-1"
step 2, use tab '-3-1' go to web site A, after page loaded, console log show "tab onload"
step 3, use tab '-3-1' go to web site B, after page loaded, console log show "tab onload"
step 4, open a new tab, console log show "onTabOpen tabId = -3-2"
step 5, drag tab '-3-1', let this tab be a new window, console log show "onTabOpen tabId = -29-1"



Actual results:

step 6, use tab '-29-1' go to web site A/B, console log not show "tab onload" anymore.


Expected results:

we should get 'load' event notification when tab page loaded.

PS. all these tab id may different with my case.

Comment 1

[Wei-Cheng Pan [:wcpan] [:wcp] [:legnaleurc] (left Mozilla)]

Can't reproduce in non-e10s mode.
Looks like a e10s-only issue.

Comment 2

[:shell escalante]

hi ettoolong, can you give us a few examples of which of your add-ons you were using it with.  helps to be able to look at the add-ons.

note to shell: High level API - need to look at this one, but conflicts with webextension work.  working on it.

Comment 3

[ettoolong]

Attachment: addon for testing (AddonSDK)

Comment 4

[ettoolong]

Attachment: webextension for testing (WebExtension)

Comment 5

[ettoolong]

Hi,

I add attachments for this AddonSDK issue.
The same function implement by AddonSDK and WebExtension, you can install them both.

test:
1. Open Firefox and open one new tab. Now we have 2 tab on first Firefox window.
2. Drag the FIRST tab and let this tab be a new window.
3. Use the second window(that create by step 2) visit any web site.

If e10s diabled: 
console log show "[WebExtension] tab onload tab.id = ..." and "[AddonSDK] tab onload tab.id = ..."

If e10s enabled:
console log only show "[WebExtension] tab onload tab.id = ..."

Comment 6

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1399562