Closed Bug 1577189 Opened 5 years ago Closed 4 years ago

Move Document::mAncestorIsLoading handling to BrowsingContext

Categories

(Core :: DOM: Core & HTML, defect, P3)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1596784
Fission Milestone M7

People

(Reporter: smaug, Assigned: smacleod)

References

(Blocks 1 open bug)

Details

mAncestorIsLoading is used to track if some ancestor is loading and if so, window's setTimeout/Interval will be deferred (this leads to better page load performance).
To make this all work in Fission, we'll need to probably track the state in BrowsingContext to handle multiple processes.

Tentatively moving all bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to the "?" triage milestone.

This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:

0ee3c76a-bc79-4eb2-8d12-05dc0b68e732

Fission Milestone: --- → ?

I initially said this should probably go in WindowContext rather than BrowsingContext, but thinking about it again, a document should probably set the flag on all of its child BrowsingContexts when its load state or an ancestor's load state changes, since the state of the BC's ancestors won't change depending on the document that's currently loaded in the BC.

Tracking for Fission Nightly (M6)

blocks fission perf

Blocks: fission-perf
Fission Milestone: ? → M6

Tracking for Fission Beta (M7) because this perf bug doesn't need to block Fission Nightly.

Fission Milestone: M6 → M7
Assignee: nobody → smacleod
Status: NEW → ASSIGNED

Haven't we been tracking this in the BrowsingContext since bug 1596784?

Is there any more work we need to do here?

Flags: needinfo?(bugs)

bug 1596784 should have been enough

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Flags: needinfo?(bugs)
Resolution: --- → FIXED
Resolution: FIXED → DUPLICATE
You need to log in before you can comment on or make changes to this bug.