Closed
Bug 1577498
Opened 5 years ago
Closed 5 years ago
Design replacement for MessageManager-based IPC
Categories
(WebExtensions :: General, task, P1)
WebExtensions
General
Tracking
(firefox72 fixed)
RESOLVED
FIXED
mozilla72
| Tracking | Status | |
|---|---|---|
| firefox72 | --- | fixed |
People
(Reporter: zombie, Assigned: zombie)
References
(Depends on 1 open bug, Blocks 4 open bugs)
Details
(Keywords: perf-alert)
Attachments
(4 files)
Currently most of our IPC is through Frame- and PP/CP MessageManagers, mostly via the MessageChannel abstraction. This probably needs a different design.
|
Assignee | |
Comment 1•5 years ago
|
||
|
||
Updated•5 years ago
|
Attachment #9089851 -
Attachment description: Bug 1577498 - ExtensionsActor and Conduits design for Fission IPC → Bug 1577498 - Part 1: ExtensionsActor and Conduits design for Fission IPC
|
|
Assignee | |
Updated•5 years ago
|
Assignee: nobody → tomica
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 3•5 years ago
|
||
|
|
Assignee | |
Comment 4•5 years ago
|
||
Pushed by tjovanovic@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0ff3601d136a
Part 1: ExtensionsActor and Conduits design for Fission IPC r=rpl,kmag
https://hg.mozilla.org/integration/autoland/rev/0f3f87195689
Part 2: Use conduits instead of messageManager in devtools panels r=rpl
https://hg.mozilla.org/integration/autoland/rev/ab07de86ac04
Part 3: Ensure actor and conduit cleanup r=rpl
|
||
Comment 6•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/0ff3601d136a
https://hg.mozilla.org/mozilla-central/rev/0f3f87195689
https://hg.mozilla.org/mozilla-central/rev/ab07de86ac04
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox72:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Pushed by tjovanovic@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c006839e639e
Part 4: Batch webRequest events to reduce IPC overhead r=rpl
|
||
Comment 8•5 years ago
|
||
Hello,
Will this fix require manual validation? If yes, please provide some steps to reproduce in order to correctly test it and also, please set the "qe-verify+" flag. Otherwise, could the "qe-verify-" flag be added? Thanks!
Flags: needinfo?(tomica)
|
|
Assignee | |
Comment 9•5 years ago
|
||
There should be no visible changes from this change, tested by all the tests we have.
Flags: needinfo?(tomica) → qe-verify-
|
||
Comment 10•5 years ago
|
||
| bugherder | ||
|
||
Updated•4 years ago
|
Keywords: perf-alert
You need to log in
before you can comment on or make changes to this bug.
Description
•