Closed Bug 1596209 Opened 5 months ago Closed 5 months ago

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

Categories

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

task
Not set

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

(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
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.