546860 - "ABORT: Instance time is dependent on itself"

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Jesse Ruderman]

Attachment: testcase (triggers fatal assertion)

###!!! ABORT: Instance time is dependent on itself: 'mLatestInstanceTime != &baseInstance', file /Users/jruderman/mozilla-central/content/smil/nsSMILTimeValueSpec.cpp, line 237

Comment 1

[Daniel Holbert [:dholbert]]

Confirmed in linux debug m-c build.

Comment 2

[Brian Birtles (:birtles)]

This code has been significantly overhauled in the patch for bug 537361 which is awaiting review. I'd be interested to see if this problems still occurs with that patch applied.

Comment 3

[Daniel Holbert [:dholbert]]

You're right, that bug's patch fixes it! No abort, no assertions -- nothing.
(I think the code touched by that patch has bitrotted a tiny bit, but I've got a bitrot-fixed version in my patch queue, with some other minor tweaks to make it stack on top of some other patches that should be landing shortly. Available here:
http://hg.mozilla.org/users/dholbert_mozilla.com/smil-patches/file/9891fe88238d/birtles_bsRefactorIntervals )

Marking dependency on that bug.

(Sorry for the long wait on the review for that bug -- I've started reviewing it, but didn't get all the way through it, and I've been meaning to finish -- I'll do that ASAP.)

Comment 4

[Daniel Holbert [:dholbert]]

This should be fixed since bug 537361 landed a few days ago, per comment 3.  I just confirmed that it's fixed in a debug build from yesterday.
  --> Resolving.