Closed
Bug 777188
Opened 12 years ago
Closed 7 years ago
Tracking: Audit usage of "parentprocessmessagemanager"
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
FIXED
People
(Reporter: cjones, Unassigned)
References
Details
See bug 776825. Apparently several folks wrote code under the mistaken assumption that there's only one content process, or that this mm somehow sends messages to the "right" content process. It doesn't, it broadcasts to all processes.
http://mxr.mozilla.org/mozilla-central/search?string=parentprocessmessagemanager
I can't really think of a case in which we should ever broadcast a message to all content processes at this level. Instead, when specific subprocesses make requests to JS services, the services need to save the .target (bug 776825 comment 4) and only reply to that.
Reporter | ||
Comment 1•12 years ago
|
||
xul-fennec has two remaining users of the parentprocessmessagemanager that we don't have to care about.
Given the fact that *every* *single* *user* *of* *ppmm* shot itself in the foot with the giant ppmm footgun, I think we need to either forbid JS proxies or fix the hell out of that interface.
Reporter | ||
Comment 2•12 years ago
|
||
Thanks Gene!
Reporter | ||
Comment 3•12 years ago
|
||
Oh sorry Gene --- my comment was probably confusing: I thought you had added a dependency I forgot to add, but now I see you removed it and added it back :). So, thanks for adding it back! ;)
Comment 4•12 years ago
|
||
(In reply to Chris Jones [:cjones] [:warhammer] from comment #3)
> Oh sorry Gene --- my comment was probably confusing: I thought you had added
> a dependency I forgot to add, but now I see you removed it and added it back
> :). So, thanks for adding it back! ;)
Cjones - Your guessing is right! I was thinking I should add the dependencies for both issues to keep track of them. ;) Btw, thanks very much for the quick review!
Updated•12 years ago
|
blocking-basecamp: --- → ?
Reporter | ||
Updated•12 years ago
|
Summary: Audit usage of "parentprocessmessagemanager" → Tracking: Audit usage of "parentprocessmessagemanager"
Updated•12 years ago
|
blocking-basecamp: ? → ---
Comment 6•7 years ago
|
||
Yeah.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(mrbkap)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•