1390831 - editor/libeditor/tests/test_bug635636.html doesn't work in e10s mode

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

The test refers gBrowser.  Therefore, it cannot run in content process.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

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

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1390831 - Make test_bug635636.html e10s-aware

test_bug655636.html refers gBrowser, however, it's available only in chrome
process and it's referred only for listening to "pageshow" event instead of
"load" event of the data URI.  So, we must be able to use "unload" event of the
previous URL instead.

Although, this testcase (even without this change) won't cause crash even if
backing out the patch for bug 635636 anymore.

Review commit: https://reviewboard.mozilla.org/r/169150/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/169150/

Comment 3

[(no longer active)]

Comment on attachment 8897869 [details]
Bug 1390831 - Make test_bug635636.html e10s-aware

https://reviewboard.mozilla.org/r/169150/#review174630

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ffe54db6e9b5
Make test_bug635636.html e10s-aware r=Ehsan

Comment 5

[Carsten Book [:Tomcat]]

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