[e10s-multi] browser_mcb_redirect.js uses CPOW that makes the test fail with multiple content processes

RESOLVED FIXED in Firefox 53

Status

()

defect
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: gkrizsanits, Assigned: mrbkap)

Tracking

(Blocks 1 bug)

unspecified
mozilla55
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox53 fixed, firefox54 fixed, firefox55 fixed)

Details

(Whiteboard: [e10s-multi:+])

Attachments

(2 attachments)

browser/base/content/test/general/browser_mcb_redirect.js | uncaught exception - Error: cross-process JS call failed at test2@chrome://mochitests/content/browser/browser/base/content/test/general/browser_mcb_redirect.js:125:3

error comes from: gTestBrowser.contentWindow.location = url;

hopefully getting rid of the CPOW will fix the issue, but I have not looked into it yet more closely.
Blocks: e10s-multi
Whiteboard: [e10s-multi:M1]
Blocks: 1301015
Whiteboard: [e10s-multi:M1]
Assignee: nobody → mrbkap
Attachment #8845134 - Flags: review?(gkrizsanits)
Attachment #8845135 - Flags: review?(gkrizsanits)
Comment on attachment 8845134 [details]
Bug 1294379 - Remove CPOWs from browser_mcb_redirect.js

https://reviewboard.mozilla.org/r/118354/#review120534

Thanks for cleaning up this mess, this looks a lot better.
Attachment #8845134 - Flags: review?(gkrizsanits) → review+
Comment on attachment 8845135 [details]
Bug 1294379 - Remove this hack.

https://reviewboard.mozilla.org/r/118356/#review120538
Attachment #8845135 - Flags: review?(gkrizsanits) → review+
Pushed by mrbkap@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5c57b1d5b3f5
Remove CPOWs from browser_mcb_redirect.js r=krizsa
https://hg.mozilla.org/integration/autoland/rev/adb505330997
Remove this hack. r=krizsa
https://hg.mozilla.org/mozilla-central/rev/5c57b1d5b3f5
https://hg.mozilla.org/mozilla-central/rev/adb505330997
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Iteration: --- → 55.1 - Mar 20
Whiteboard: [e10s-multi:+]
You need to log in before you can comment on or make changes to this bug.