Closed Bug 1596209 Opened 6 years ago Closed 6 years ago

Remove GetBindingParent (dealing with NAC and Shadow DOM separately).

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
task
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla72
Tracking Flags:
Tracking Status
firefox72 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1596209 - Remove nsIContent::GetBindingParent. r=smaug
47 bytes, text/x-phabricator-request
Details | Review

(And use explicit shadow dom checks where needed)

Type: defect → task

So NAC would be the only use for bindingparent? If so, could we figure out how to get rid of it altogether?

Yeah, let's try to do that.

Assignee: nobody → emilio
Depends on: 1596391
Summary: See if it's worth to remove mBindingParent shadow DOM usage → Remove GetBindingParent (dealing with NAC and Shadow DOM separately).
Depends on: 1596445
Depends on: 1596451
Depends on: 1596455
Depends on: 1596463
Depends on: 1596465
Depends on: 1596474
Depends on: 1596502
Depends on: 1596506
Depends on: 1596511
Depends on: 1596768
Blocks: 1596800

It's unused after all the previous patches.

Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0478d1bf6e32 Remove nsIContent::GetBindingParent. r=smaug
Backout by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/autoland/rev/bbbe694dbff2 Backed out changeset 0478d1bf6e32 on request from emilio. CLOSED TREE
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dd4a4052989c Remove nsIContent::GetBindingParent. r=smaug

https://hg.mozilla.org/mozilla-central/rev/dd4a4052989c

Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox72: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: