1256513 - Convert the test for bug 527935 into a browser-chrome test that is enabled on e10s

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect)

Comment 1

[(no longer active)]

Attachment: Convert the test for bug 527935 into a browser-chrome test that is enabled on e10s

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 8730483 [details] [diff] [review]
Convert the test for bug 527935 into a browser-chrome test that is enabled on e10s

This is missing the added browser_bug527935.js file.

Comment 3

[(no longer active)]

Attachment: Convert the test for bug 527935 into a browser-chrome test that is enabled on e10s

Sorry for forgetting to add the file previously!

Comment 4

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 8730727 [details] [diff] [review]
Convert the test for bug 527935 into a browser-chrome test that is enabled on e10s

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

::: editor/libeditor/tests/browser_bug527935.js
@@ +12,5 @@
> +    }
> +    SpecialPowers.addAutoCompletePopupEventListener(window, "popupshowing", listener);
> +
> +    yield ContentTask.spawn(aBrowser, {}, function*() {
> +      var window = content.window.wrappedJSObject;

Do you need the .wrappedJSObject here and below?

Comment 5

[(no longer active)]

Yes, in order to make the loadPromise bit work.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/97aba539ecb9

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/97aba539ecb9