1231695 - Make browser_bug593387.js pass in e10s

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Blake Kaplan (:mrbkap) (inactive)]

This test uses browser.contentWindow on a remote browser. It's very straightforward.

Comment 1

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Don't use contentWindow on remote browsers

Comment 2

[Andrew McCreight [:mccr8]]

Comment on attachment 8697295 [details] [diff] [review]
Don't use contentWindow on remote browsers

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

::: dom/base/test/browser_bug593387.js
@@ -27,4 @@
>    var frame = newBrowser.contentDocument.createElement("iframe");
>    frame.src = "http://mochi.test:8888/tests/dom/base/test/file_x-frame-options_page.sjs?testid=deny&xfo=deny";
> -  frame.addEventListener("load", function() {
> -    frame.removeEventListener("load", arguments.callee, true);

eww

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d9f0f6221e06ccea601772206c8f602e1255651c
Bug 1231695 - Don't use contentWindow on remote browsers. r=mccr8

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d9f0f6221e06

Comment 5

[Blake Kaplan (:mrbkap) (inactive)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b09992a0dccd

Comment 6

[Blake Kaplan (:mrbkap) (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/9e1c3b1f311cfa17e98cb09b3e29b197a1b51f07
Bug 1231695 - Enable this test. r=mccr8

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/9e1c3b1f311c