Closed Bug 678847 Opened 9 years ago Closed 9 years ago

"ABORT: Update current interval recursion depth exceeded threshold" with <svg:animate>

Categories

(Core :: SVG, defect)

defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla9

People

(Reporter: jruderman, Assigned: birtles)

References

(Blocks 1 open bug)

Details

(Keywords: assertion, testcase)

Attachments

(5 files, 1 obsolete file)

###!!! ABORT: Update current interval recursion depth exceeded threshold: 'PR_FALSE', file content/smil/nsSMILTimedElement.cpp, line 1949

Does not crash in opt.
Attached file stack trace
Assignee: nobody → birtles
Status: NEW → ASSIGNED
Crash test--same as attachment 553005 [details] but packed up as a crash test
Attachment #555939 - Flags: review?(dholbert)
Attached patch Part 2 - Proposed patch v1a (obsolete) — Splinter Review
Proposed patch.
Attachment #555940 - Flags: review?(dholbert)
Alternative patch just for reference
Comment on attachment 555940 [details] [diff] [review]
Part 2 - Proposed patch v1a

I need to rethink this a bit
Attachment #555940 - Attachment is obsolete: true
Attachment #555940 - Flags: review?(dholbert)
It's generally not a good idea to file patches as you're going out the door but this should be better than the last attempt anyway.
Attachment #555945 - Flags: review?(dholbert)
OS: Mac OS X → All
Hardware: x86_64 → All
Attachment #555939 - Flags: review?(dholbert) → review+
Comment on attachment 555945 [details] [diff] [review]
Part 2 - Proposed patch v1b

Looks good.
Attachment #555945 - Flags: review?(dholbert) → review+
http://hg.mozilla.org/mozilla-central/rev/7a01e603335d
http://hg.mozilla.org/mozilla-central/rev/246f74de583c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.