20946 2008-09-19 12:21:24 -0700 REGRESSION: Invalid CSS @media query should be ignored but isn't 2009-01-06 00:06:12 -0800 1 1 1 Unclassified WebKit CSS 528+ (Nightly build) Mac OS X 10.5 RESOLVED INVALID P1 Major --- 1 c.petersen87 webkit-unassigned apapirovski webkit oldest_to_newest 91801 0 c.petersen87 2008-09-19 12:21:24 -0700 * SUMMARY With Webkit NB r36640 , a invalid CSS @media query should be ignored but isn't. This problem was originally found at http://store.iomega.com/section?SID=f2e44e869a9564289240548cef0a492cc48:4760&secid=40299. I have reduced it down with Tim H to a simple test case. <style> .test { background-color: green; width: 20px; height: 20px; } @media screen { /* Any style rules after this will be ignored. */ .test { display: none; } </style> <div class="test"></div> * STEPS TO REPRODUCE 1. Open the attached file "iomega.html" in Safari 3.1. 2. Notice a green square (< DIV> ) appears on this page 3. Now, open this same test case in TOT. Notice the green square doesn't appear on the page at all. The display: none; rule is being processed even thought it shouldn't be ( since the @media query is not valid) 91802 1 23577 c.petersen87 2008-09-19 12:22:09 -0700 Created attachment 23577 Reduced test case 91803 2 c.petersen87 2008-09-19 12:22:53 -0700 <rdar://problem/6232970> 102271 3 apapirovski 2008-12-11 13:31:31 -0800 The @media query is valid! See here http://www.w3.org/TR/CSS21/syndata.html#parsing-errors - specifically the section about "Unexpected end of style sheet". 102272 4 apapirovski 2008-12-11 13:33:48 -0800 More information: Firefox 3 gets this wrong, Opera 9.5 gets it right. 104538 5 webkit 2009-01-06 00:06:12 -0800 (In reply to comment #3) > The @media query is valid! See here > http://www.w3.org/TR/CSS21/syndata.html#parsing-errors - specifically the > section about "Unexpected end of style sheet". > I agree with Anatoli. The reported style sheet is absolutely correct. (In reply to comment #4) > More information: Firefox 3 gets this wrong, Opera 9.5 gets it right. > Firefox 3.1 beta 2 has this bug fixed. I close this bug as INVALID since it doesn't describe a real bug. 23577 2008-09-19 12:22:09 -0700 2008-09-19 12:22:09 -0700 Reduced test case iomega.html text/html 203 c.petersen87 PHN0eWxlPgoudGVzdCB7CgliYWNrZ3JvdW5kLWNvbG9yOiBncmVlbjsKCXdpZHRoOiAyMHB4OwoJ aGVpZ2h0OiAyMHB4Owp9CgpAbWVkaWEgc2NyZWVuIHsKCi8qIEFueSBzdHlsZSBydWxlcyBhZnRl ciB0aGlzIHdpbGwgYmUgaWdub3JlZC4gKi8KCi50ZXN0IHsKCWRpc3BsYXk6IG5vbmU7Cn0KPC9z dHlsZT4KCjxkaXYgY2xhc3M9InRlc3QiPjwvZGl2Pgo=