1827856 - DevTools don't observe mutations to UA widgets correctly.

Status: RESOLVED FIXED

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

Description

[Emilio Cobos Álvarez (:emilio)]

STR:

• Set devtools.inspector.showAllAnonymousContent=true
• Open data:text/html,<video controls>
• Inspect the play/pause button.
• Click it

ER:

• The inspector view updates to show the changed attributes.

AR:

• It doesn't. It seems they're using MutationObserver but the listeners are not getting called properly.

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1827856 - Speed up GetClosestAnonymousSubtreeRoot for UA widgets. r=smaug

Depends on D175341

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1827856 - Improve docs of IsInSameAnonymousTree. r=smaug

Depends on D175353

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1827856 - Misc cleanups to nsDOMMutationObserver. r=smaug

Depends on D175354

Comment 4

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1827856 - Allow chrome code to properly observe UA widgets / NAC. r=smaug

Depends on D175355

Comment 5

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1827856 - Remove nativeAnonymousChildList observers. r=smaug

You let me know if this seems appealing to you :)

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/632a4846f98e
Speed up GetClosestAnonymousSubtreeRoot for UA widgets. r=smaug
https://hg.mozilla.org/integration/autoland/rev/63a815a0c38c
Improve docs of IsInSameAnonymousTree. r=smaug
https://hg.mozilla.org/integration/autoland/rev/794e1e64e3ab
Misc cleanups to nsDOMMutationObserver. r=smaug

Comment 7

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/632a4846f98e
https://hg.mozilla.org/mozilla-central/rev/63a815a0c38c
https://hg.mozilla.org/mozilla-central/rev/794e1e64e3ab

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2745e11153d9
Allow chrome code to properly observe UA widgets / NAC. r=smaug,devtools-reviewers,nchevobbe

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9c6e31cf2855
Remove nativeAnonymousChildList observers. r=smaug,credential-management-reviewers,devtools-reviewers,sgalich,nchevobbe

Comment 10

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/2745e11153d9

Comment 11

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/9c6e31cf2855