1189550 - In parent process, cpmm.initialProcessData should be same as ppmm.initialProcessData

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, defect)

Description

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

Attachment: patch

The typical way to use initialProcessData is to set it on the ppmm and read it from the cpmm. That doesn't work for the cpmm in the chrome process. This patch fixes that.

I also threw in a patch for bug 1184115. It makes it so that ppmm is undefined in the child process. I don't think there's any reason you would ever want it there.

Comment 1

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

Comment on attachment 8641338 [details] [diff] [review]
patch

always {} with if.

Comment 2

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b821b18a1bfb

Comment 3

[Carsten Book [:Tomcat]]

sorry had to back this out in https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=4e3821b236f9 since one of the 2 changes seems to have caused:

for causing test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=12405582&repo=mozilla-inbound

Comment 4

[Pulsebot]

Backout:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5e2d1a51d609

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/502c196722eb

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/502c196722eb