Closed Bug 1098666 Opened 7 years ago Closed 7 years ago

[e10s] Make cpow.navigator work with site-specific navigator fixup

Categories

(Core :: DOM: Content Processes, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla36
Tracking Status
e10s ? ---

People

(Reporter: mrbkap, Assigned: mrbkap)

References

Details

Attachments

(2 files)

Current cpow.navigator.userAgent (in the child) calls into JS, which does a messagemanager.sendSyncMessage to the parent. Because we're already handling a high CPOW message, this aborts. For the moment, the best we can do is switch to using sendRpcMessage. khuey was complaining that we should weaken our assertions when there's a CPOW on the stack, but I don't want to deal with that here.

This was causing Tobias' nightly to crash in e10s mode because the "Blur" or "DoNotTrackMe" extension runs into this.
Attached patch patch v1Splinter Review
I'll attach a testcase for this tomorrow.
Attachment #8522617 - Flags: review?(wmccloskey)
Comment on attachment 8522617 [details] [diff] [review]
patch v1

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

Thanks. Any chance you can add a test for this to test_cpows?

I don't think we should weaken the assertions. Each time we convert sendSyncMessage to sendRpcMessage, we need to be sure we're not breaking some ordering requirement. Weakening the assertions is essentially the same as converting them all without considering ordering.
Attachment #8522617 - Flags: review?(wmccloskey) → review+
Attached patch Add a test.Splinter Review
Attachment #8523222 - Flags: review?(wmccloskey)
Comment on attachment 8523222 [details] [diff] [review]
Add a test.

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

Thanks!
Attachment #8523222 - Flags: review?(wmccloskey) → review+
https://hg.mozilla.org/mozilla-central/rev/6039cbdd69a9
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.