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

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, defect)

Description

[Blake Kaplan (:mrbkap) (inactive)]

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.

Comment 1

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: patch v1

I'll attach a testcase for this tomorrow.

Comment 2

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

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.

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Add a test.

Comment 4

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Comment on attachment 8523222 [details] [diff] [review]
Add a test.

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

Thanks!

Comment 5

[Blake Kaplan (:mrbkap) (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6039cbdd69a9

Thanks for the fast reviews!

Comment 7

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/6039cbdd69a9