Closed Bug 1272873 Opened 8 years ago Closed 7 years ago

sdk/tabs not fire load event

Categories

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

defect

Tracking

(e10s+)

RESOLVED INCOMPLETE
Tracking Status
e10s + ---

People

(Reporter: ettoolong, Unassigned, NeedInfo)

References

Details

(Whiteboard: [tabs, sdk] triaged impactse10s)

Attachments

(2 files)

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.
Can't reproduce in non-e10s mode.
Looks like a e10s-only issue.
Whiteboard: [tabs, sdk]
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.
Flags: needinfo?(sescalante)
Flags: needinfo?(ettoolong)
Priority: -- → P2
Whiteboard: [tabs, sdk] → [tabs, sdk] triaged
Flags: needinfo?(sescalante)
Whiteboard: [tabs, sdk] triaged → [tabs, sdk] triaged impactse10s
Attached file addon for testing (AddonSDK) (deleted) —
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 = ..."
Blocks: triaged-tracking
No longer blocks: e10s-addons
https://bugzilla.mozilla.org/show_bug.cgi?id=1399562
Status: UNCONFIRMED → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: