Tracking: Audit usage of "parentprocessmessagemanager"

NEW
Unassigned

Status

()

Core
General
5 years ago
5 years ago

People

(Reporter: cjones, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

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.
Depends on: 777196
Depends on: 777198
Depends on: 777200
Depends on: 777202
Depends on: 777203
Depends on: 777204
Depends on: 777206
Depends on: 777207
Depends on: 777208
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.
No longer depends on: 777207
Thanks Gene!
Depends on: 777207
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! ;)
(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!
blocking-basecamp: --- → ?
Summary: Audit usage of "parentprocessmessagemanager" → Tracking: Audit usage of "parentprocessmessagemanager"
blocking-basecamp: ? → ---
Depends on: 783392

Updated

5 years ago
Depends on: 784766
You need to log in before you can comment on or make changes to this bug.