1270119 - Update browser_localStorage_privatestorageevent.js to run on e10s

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, defect)

Comment 1

[Nika Layzell [:nika] (ni? for response)]

Attachment: Rewrite browser_localStorage_privatestoragevent.js to run on e10s

Comment 2

[Josh Matthews [:jdm]]

Comment on attachment 8748652 [details] [diff] [review]
Rewrite browser_localStorage_privatestoragevent.js to run on e10s

Review of attachment 8748652 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/tests/browser/browser_localStorage_privatestorageevent.js
@@ +19,4 @@
>  
> +  var privTab = privWin.gBrowser.addTab(URL);
> +  yield BrowserTestUtils.browserLoaded(privWin.gBrowser.getBrowserForTab(privTab));
> +  var privBrowser = gBrowser.getBrowserForTab(privTab);

I find this suspect; why isn't this using privWin?

@@ +22,5 @@
> +  var privBrowser = gBrowser.getBrowserForTab(privTab);
> +
> +  var pubTab = pubWin.gBrowser.addTab(URL);
> +  yield BrowserTestUtils.browserLoaded(pubWin.gBrowser.getBrowserForTab(pubTab));
> +  var pubBrowser = gBrowser.getBrowserForTab(pubTab);

I find this suspect; why isn't this using pubWin?

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3fa8799f00e7

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/3fa8799f00e7