Closed Bug 460209 Opened 11 years ago Closed 4 years ago

"ASSERTION: media list must be nonempty" strikes back

Categories

(Core :: CSS Parsing and Computation, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: jruderman, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: assertion, testcase)

Attachments

(1 file)

Attached file testcase
###!!! ASSERTION: media list must be nonempty: 'media->Count() != 0', file /Users/jruderman/central/layout/style/nsCSSParser.cpp, line 1778

This is the NS_ASSERTION line that was left intact in bug 449171.
I suspect this assertion is no longer valid after bug 454226; I think what happens now is that '@import "404.css" s x' parses successfully, but has a media query list with zero valid queries (but, since it had invalid queries, is not equivalent to 'all').
That seems reasonable.  Want to remove the assert, then?
Testcase works without asserting now.
Status: NEW → RESOLVED
Closed: 4 years ago
Flags: in-testsuite+
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.