1259292 - pageMod worker tab becomes false, tab.id = undefined

Status: RESOLVED INCOMPLETE

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

Description

[tim]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.116 Safari/537.36

Steps to reproduce:

create worker using pageMod, open up some pages, then drag out one of the pages

http://stackoverflow.com/questions/36190084/how-do-i-keep-track-of-workers-tabs-via-pagemod-or-worker-tab-lost


Actual results:

the worker corresponding to the dragged out page gets .tab = false, so that tab.id = undefined. This makes it not possible to find in an activeTabs search 


Expected results:

the worker gets an updated tab and tab.id

Comment 1

[tim]

I have the same problem in nightly 48.0a1 (2016-03-16).

```
console.log: capturemoz: pipeWorker: waiting for connection...
console.log: capturemoz: pipeWorker: connected.
console.log: capturemoz: attaching pageWorker in tab.id -5-1
console.log: capturemoz: showing page in tab.id -5-1
console.log: capturemoz: attaching pageWorker in tab.id -5-2
console.log: capturemoz: showing page in tab.id -5-2
<at this point I drag out the window>
console.log: capturemoz: hiding page in tab.id undefined
console.log: capturemoz: showing page in tab.id undefined
```

tab.id is undefined, and tab = false

Comment 2

[Loic]

Any testcase?

Comment 3

[tim]

Make a new jpm project, and replace index.js with this. Then make a blank worker.js. Use jpm run to start Firefox and go to Google in two tabs. Then drag out one of the tabs. https://github.com/tofutim/pagepulla


var self = require("sdk/self");
var pageMod = require("sdk/page-mod");

exports.main = function(options, callbacks) {
  pageMod.PageMod({
	  include: "*",
	  contentScriptWhen: "ready",
	  contentScriptFile: ["./worker.js"],
	//  attachTo: ["existing", "top"],
	  onAttach: function(pageWorker) {
			pageWorker.id = Date.now();
		  console.log('attaching pageWorker', pageWorker.id,'in tab.ids',pageWorker.tab.id);
			pageWorker.on('pageshow', function() {
				console.log('showing pageWorker', pageWorker.id,'in tab.ids',pageWorker.tab.id);
			});
			pageWorker.on('pagehide', function() {
				console.log('hiding pageWorker', pageWorker.id,'in tab.ids',pageWorker.tab.id);
			});
	    pageWorker.on('detach', function() {
				console.log('detaching pageWorker', pageWorker.id,'in tab.ids',pageWorker.tab.id);
	      detachPageWorker(pageWorker);
	    });
	  }
  });
}

Comment 4

[tim]

Thanks for taking a look Loic. I updated the github repo with photos. Pls take a look and let me know if you need anything else.

Comment 5

[Andrew Overholt [:overholt]]

I don't think tab.id is a DOM thing (but I could be wrong).

Comment 6

[Firefox Bug Husbandry Bot]

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