Closed
Bug 757700
Opened 13 years ago
Closed 13 years ago
"ABORT: Unknown modification type" with mutation observer, SVG transform
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: jruderman, Assigned: smaug)
References
Details
(Keywords: assertion, testcase)
Attachments
(3 files)
###!!! ABORT: Unknown modification type.: 'aModType == nsIDOMMutationEvent::MODIFICATION', file content/svg/content/src/nsSVGGraphicElement.cpp, line 174
I think the problem is that "modType" is used uninitialized in nsGenericElement::SetAttr.
Reporter | ||
Comment 1•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → bugs
Assignee | ||
Comment 2•13 years ago
|
||
See http://hg.mozilla.org/mozilla-central/diff/83dce280d871/content/base/src/nsGenericElement.cpp#l1.199
Attachment #626351 -
Flags: review?(jonas)
Comment on attachment 626351 [details] [diff] [review]
patch
Review of attachment 626351 [details] [diff] [review]:
-----------------------------------------------------------------
Definitely not related to this bug, but I wonder if it's the right decision to fire mutation observers when an attribute is set to its old value. Last time I measured it was quite common for attributes to be set to a value they already had.
Anyhow, not related to this bug, and if you feel that it's the right thing to fire mutation observers for this situation, then feel free to ignore this comment.
Attachment #626351 -
Flags: review?(jonas) → review+
Assignee | ||
Comment 4•13 years ago
|
||
(In reply to Jonas Sicking (:sicking) from comment #3)
> Comment on attachment 626351 [details] [diff] [review]
> patch
>
> Review of attachment 626351 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> Definitely not related to this bug, but I wonder if it's the right decision
> to fire mutation observers when an attribute is set to its old value. Last
> time I measured it was quite common for attributes to be set to a value they
> already had.
Well, per the current spec, mutationobserver should be fired.
Assignee | ||
Comment 5•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Target Milestone: --- → mozilla15
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•