1428784 - Consider using nsIMutationObserver and its AttributeWillChange callback for CE attribute change notifications

Status: RESOLVED WONTFIX

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

Description

[Edgar Chen [:edgar]]

+++ This bug was initially created as a follow-up of Bug #1428244 comment #9 +++

Consider using nsIMutationObserver and its AttributeWillChange callback for CE attribute change notifications.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

So my idea here is that if code isn't using CE, it would get the fastest possible code paths, since there wouldn't be any CE checks.
But I'm not sure if this would then slow down CE implementation.

Comment 2

[Edgar Chen [:edgar]]

And it could probably also fix bug 1428246, since the additional SetAttrAndNotify() call called with aNotify=false.

Comment 3

[Edgar Chen [:edgar]]

(In reply to Edgar Chen [:edgar] from comment #2)
> And it could probably also fix bug 1428246, since the additional
> SetAttrAndNotify() call called with aNotify=false.
  ^^^^^^^^^^^^^^^^^^
  I meant to say SetInlineStyleDeclaration().

Comment 4

[Edgar Chen [:edgar]]

It will be good if we could have tests for parser case first.

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

this wouldn't really solve the style attribute issue, since if we do this, then attribute changes coming from the parser aren't triggered.

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

This is actually hard because of how we handle attributes in parser.
And the fix for bug 1428246 is to make style attribute handling sane.
Closing.