Closed Bug 1647431 Opened 3 months ago Closed 3 months ago

Fix use of GetTopLevelContentDocument in Document::ReportShadowDOMUsage

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla80
Fission Milestone M7
Tracking Status
firefox80 --- fixed

People

(Reporter: kmag, Assigned: smaug)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

GetTopLevelContentDocument returns null when the top-level document is in a different process, or there are cross-process interstitials between it and the current document. This means that for these frames, shadow DOM usage is never reported.

Severity: -- → S3
Fission Milestone: --- → ?

Olli, how critical would this be for Fission?

Flags: needinfo?(bugs)

Not critical. This is just a rather nice helper when debugging web sites possibly using Shadow DOM.

Let me fix this.

Assignee: nobody → bugs
Flags: needinfo?(bugs)
Fission Milestone: ? → M7
Severity: S3 → N/A
Status: NEW → ASSIGNED
Type: defect → task
Priority: -- → P4
Priority: P4 → P3
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3d2ddbf83ab9
make ReportShadowDOMUsage Fission compatible, r=edgar
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.