Last Comment Bug 675722 - Reduce severity of "Expecting to be paused for pagehide before disconnect" SMIL warning from ABORT to ASSERTION, since we have code that triggers it
: Reduce severity of "Expecting to be paused for pagehide before disconnect" SM...
Status: RESOLVED FIXED
[inbound]
:
Product: Core
Classification: Components
Component: SVG (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla8
Assigned To: Daniel Holbert [:dholbert]
:
: Jet Villegas (:jet)
Mentors:
Depends on:
Blocks: 671976
  Show dependency treegraph
 
Reported: 2011-08-01 13:32 PDT by Daniel Holbert [:dholbert]
Modified: 2011-08-04 12:45 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
fix: s/ABORT_IF_FALSE/ASSERTION/, and add assertion in ~nsDocument (1.93 KB, patch)
2011-08-01 14:05 PDT, Daniel Holbert [:dholbert]
bzbarsky: review+
Details | Diff | Splinter Review

Description Daniel Holbert [:dholbert] 2011-08-01 13:32:14 PDT
Bug 654015 added an ABORT_IF_FALSE to be sure that we always fire PageHide on the animation controller before the document is destroyed.

However, we have cases where that assertion doesn't hold up (which are almost certainly bugs, but which nonetheless trip this ABORT_IF_FALSE and keep fuzzers from progressing).  e.g. bug 670324 and bug 654015 (the latter of which was the bug that added this assertion; but, had the assertion been around before that bug was filed, its testcase would've tripped it).

I'm not sure what the best solution to bug 670324 is, and I don't want to commit to immediately fixing all similar bugs in the future, so I think it's best to downgrade this assertion to simply "assertion-level" rather than "abort-level".  It's not a particularly scary assertion, from the perspective of the nsSMILAnimationController -- I only made it "abort-level" in the first place because I (mistakenly) thought it was a condition that could absolutely never happen.
Comment 1 Daniel Holbert [:dholbert] 2011-08-01 14:05:44 PDT
Created attachment 549912 [details] [diff] [review]
fix: s/ABORT_IF_FALSE/ASSERTION/, and add assertion in ~nsDocument

This patch reduces the assertion severity, and it also adds a related assertion in ~nsDocument to catch this earlier, per bug 670324 comment 7. (The second assertion will hopefully make it more obvious that these assertion-failures will tend to to be document-show/hide issues rather than SMIL-related issues.)
Comment 2 Boris Zbarsky [:bz] (still a bit busy) 2011-08-01 20:59:54 PDT
Comment on attachment 549912 [details] [diff] [review]
fix: s/ABORT_IF_FALSE/ASSERTION/, and add assertion in ~nsDocument

r=me
Comment 3 Daniel Holbert [:dholbert] 2011-08-02 16:13:42 PDT
http://hg.mozilla.org/integration/mozilla-inbound/rev/4bf9b8abb862
Comment 4 Marco Bonardo [::mak] 2011-08-03 02:22:21 PDT
http://hg.mozilla.org/mozilla-central/rev/4bf9b8abb862

Note You need to log in before you can comment on or make changes to this bug.