1015763 - tabs/utils getTabForContentWindow() throws with e10s enabled

Status: RESOLVED FIXED

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

Description

[Tomislav Jovanovic :zombie]

the `browser` in the child process on line 263 in [1] doesn't have an `ownerDocument`. we could probably just iterate over tabs and compare `contentWindow`s, like we already do for Fennec.

https://github.com/mozilla/addon-sdk/blob/master/lib/sdk/tabs/utils.js#L263

Comment 1

[Tomislav Jovanovic :zombie]

Attachment: Link to Github pull-request: https://github.com/mozilla/addon-sdk/pull/1498

and i did just that. 

no new tests, as this is to fix already existing tab tests when running with e10s enabled.

Matteo, can you please run the tests on Fennec, as i reduced the two methods into one?

Comment 2

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/23a07371d5113721fb221c65a1191845f2d03ef5
bug 1015763 - e10s getTabForContentWindow throws

https://github.com/mozilla/addon-sdk/commit/c675ffdcece90e2583f6a96f7c153b9d4a689aa9
Merge pull request #1498 from zombie/1015763-getTabForContentWindow

bug 1015763 - getTabForContentWindow throws with e10s, r=@ZER0