680413 - Allow using child process message manager in chrome process

Status: RESOLVED FIXED

(Core :: IPC, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

...this way certain scripts can run also in non-e10s builds, and
in-process-frame-scripts can also use process mm.

Uploaded to try.

Comment 1

[Benjamin Smedberg]

What needs this? I really want to discourage use of the process message manager and I don't want to take this unless the alternative is really messy code in some dependency.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

If any frame mm script uses process mm, it would be hard to use the same code in-process mm.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

So, the patch is about having consistent APIs.

Comment 4

[Jez Ng [:int3]]

Now that Session Restore is using the process message manager to handle data sent on unload, we need this patch to ensure that the same code works both when e10s is enabled and disabled.

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

https://hg.mozilla.org/mozilla-central/rev/9a72a10e8bb5

Comment 6

[Masatoshi Kimura [:emk]]

> +  PRBool isChrome = IsChromeProcess();
Should be bool.