"ASSERTION: media list must be nonempty" strikes back

RESOLVED WORKSFORME

Status

()

Core
CSS Parsing and Computation
RESOLVED WORKSFORME
10 years ago
3 years ago

People

(Reporter: Jesse Ruderman, Unassigned)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
x86
Mac OS X
assertion, testcase
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
Created attachment 343354 [details]
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
Last Resolved: 3 years ago
Flags: in-testsuite+
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.