Closed Bug 1577498 Opened 1 year ago Closed 10 months ago

Design replacement for MessageManager-based IPC

Categories

(WebExtensions :: General, task, P1)

task

Tracking

(firefox72 fixed)

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: zombie, Assigned: zombie)

References

(Depends on 2 open bugs, Blocks 4 open bugs)

Details

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.

Blocks: 1580774
Blocks: 1583484
Depends on: 1578420
Attachment #9089851 - Attachment description: Bug 1577498 - ExtensionsActor and Conduits design for Fission IPC → Bug 1577498 - Part 1: ExtensionsActor and Conduits design for Fission IPC
Depends on: 1587058
Assignee: nobody → tomica
Depends on: 1595186
Blocks: 1595186
No longer depends on: 1595186
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
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Regressions: 1597906
See Also: → 1597965
Pushed by tjovanovic@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c006839e639e
Part 4: Batch webRequest events to reduce IPC overhead r=rpl

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)

There should be no visible changes from this change, tested by all the tests we have.

Flags: needinfo?(tomica) → qe-verify-
Depends on: 1605098
Blocks: 1398672
You need to log in before you can comment on or make changes to this bug.