Closed Bug 1278457 Opened 9 years ago Closed 7 years ago

"Assertion failure: aTextNode && aTextNode->IsNodeOfType(nsINode::eTEXT)" with colgroup, comment node

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox-esr52 --- wontfix

People

(Reporter: jruderman, Assigned: xidorn)

References

Details

(Keywords: assertion, testcase)

Attachments

(2 files)

Attached file testcase
###!!! ASSERTION: shouldn't waste time creating style contexts for comments and processing instructions: 'aContent->IsNodeOfType(nsINode::eTEXT)', file layout/base/nsCSSFrameConstructor.cpp, line 4969 Assertion failure: aTextNode && aTextNode->IsNodeOfType(nsINode::eTEXT), at layout/style/nsStyleSet.cpp:1756
Attached file stack
Fix range: INFO: First good revision: c63279dd3ec425d3fa8aa6e55948792b4ae608d4 INFO: Last bad revision: 590858b739ab6caaae7ceade33dda746ea6360ad INFO: Pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=590858b739ab6caaae7ceade33dda746ea6360ad&tochange=c63279dd3ec425d3fa8aa6e55948792b4ae608d4 Xidorn, is it worth landing this testcase as a crashtest or do the other ones from bug 1308848 already have this one covered?
Assignee: nobody → xidorn+moz
Status: NEW → RESOLVED
Has Regression Range: --- → no
Closed: 7 years ago
Depends on: 1308848
Flags: needinfo?(xidorn+moz)
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
I don't think it's worth. The crashtests added there should be good enough.
Flags: needinfo?(xidorn+moz)
Flags: in-testsuite? → in-testsuite-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: