Closed Bug 1646510 Opened 4 years ago Closed 4 years ago

[meta] Audit uses of nsContentUtils::GetCrossDocParentNode

Categories

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

task

Tracking

()

RESOLVED FIXED
89 Branch
Fission Milestone M7
Tracking Status
firefox89 --- fixed

People

(Reporter: kmag, Assigned: kmag)

References

(Blocks 1 open bug)

Details

(Keywords: meta)

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1642433 +++

Similar to uses of nsIDocShellTreeItem, many/most of the callers of these methods will probably do the wrong thing under Fission with remote frames.

Depends on: 1647407
Depends on: 1647412
Depends on: 1647416
Depends on: 1647421

The only remaining usage I see is https://searchfox.org/mozilla-central/source/dom/events/EventStateManager.cpp#3083 in NodeAllowsClickThrough().

Fission Milestone: --- → ?

(In reply to Neha Kochar [:neha] from comment #1)

The only remaining usage I see is https://searchfox.org/mozilla-central/source/dom/events/EventStateManager.cpp#3083 in NodeAllowsClickThrough().

NodeAllowsClickThrough() looks for XUL element which never exists in content process anyway. If this is broken, then it's also broken currently with e10s. Nothing more to do here so we'll close this.

Status: NEW → RESOLVED
Fission Milestone: ? → M7
Closed: 4 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Moves its last remaining (parent-process-only) caller to use a local static
copy instead.

Pushed by maglione.k@gmail.com: https://hg.mozilla.org/integration/autoland/rev/4a43c293dc05 Remove deprecated nsContentUtils::GetCrossDocParentNode. r=nika
Status: REOPENED → RESOLVED
Closed: 4 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: