Closed Bug 1598775 Opened 3 months ago Closed 3 months ago

Ensure postMessage related optimizations during page load work also in Fission

Categories

(Core :: DOM: Navigation, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla72
Fission Milestone M7
Tracking Status
firefox72 --- fixed

People

(Reporter: smaug, Assigned: smaug)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

window.postMessage uses lower priority queue during page load.
We should ensure the setup works also with Fission.

Ah, it added https://searchfox.org/mozilla-central/rev/3483fb259b4edbe4594cfcc3911db97d5441b67d/dom/base/TabGroup.cpp#258
and TabGroups are being replaced with BrowsingContextGroup.

Priority: -- → P3
Component: DOM: Core & HTML → DOM: Navigation
Fission Milestone: --- → M7
Assignee: nobody → bugs

Bug 1534012 added a test for postMessage during load and that revealed that mLoading flag isn't always updated
soon enough, so add BrowsingContext::IsLoading which checks also possible local loading status.

Depends on D54754

Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1c9bf0994ad0
Ensure postMessage related optimizations during page load work also in Fission, r=farre
https://hg.mozilla.org/integration/autoland/rev/76d464c781b2
Ensure postMessage related optimizations during page load work also in Fission, part 2, r=farre
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.