Closed
Bug 1330492
Opened 7 years ago
Closed 7 years ago
Assertion failure: mDocument->GetReadyStateEnum() == nsIDocument::READYSTATE_LOADING (Bad readyState)
Categories
(Core :: XSLT, defect, P3)
Core
XSLT
Tracking
()
RESOLVED
FIXED
mozilla58
People
(Reporter: truber, Assigned: RyanVM)
References
(Blocks 1 open bug)
Details
(Keywords: assertion, testcase)
Attachments
(3 files)
The attached testcase causes the assertion below in mozilla-central rev 63ad56438630 Assertion failure: mDocument->GetReadyStateEnum() == nsIDocument::READYSTATE_LOADING (Bad readyState), at /home/worker/workspace/build/src/dom/xslt/xslt/txMozillaTextOutput.cpp:86
Reporter | ||
Comment 1•7 years ago
|
||
Comment 3•7 years ago
|
||
Yeah, the assert should change, if mObserver is null then |mDocument->GetReadyStateEnum()| should be |nsIDocument::READYSTATE_INTERACTIVE| instead of |nsIDocument::READYSTATE_LOADING|.
Blocks: 1205163
Updated•7 years ago
|
Flags: needinfo?(peterv)
Priority: -- → P3
Updated•7 years ago
|
Assignee | ||
Comment 4•7 years ago
|
||
Per comment 3. I've confirmed that the crashtest fails without the patch and passes with it.
Assignee: nobody → ryanvm
Attachment #8911434 -
Flags: review?(peterv)
Updated•7 years ago
|
Attachment #8911434 -
Flags: review?(peterv) → review+
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/26a6bdc37e04 Check for nsIDocument::READYSTATE_INTERACTIVE if mObserver is null. r=peterv
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/26a6bdc37e04
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Assignee | ||
Updated•7 years ago
|
status-firefox55:
--- → wontfix
status-firefox56:
--- → wontfix
status-firefox-esr52:
--- → wontfix
Flags: in-testsuite? → in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•