Closed
Bug 784405
Opened 12 years ago
Closed 12 years ago
firefox accepts invalid url() for at least background-image
Categories
(Core :: CSS Parsing and Computation, defect)
Tracking
()
RESOLVED
INVALID
People
(Reporter: cobexer, Unassigned)
Details
Attachments
(1 file)
|
978 bytes,
text/html
|
Details |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20100101 Firefox/14.0.1
Build ID: 2012071300
Steps to reproduce:
<!doctype html>
<html>
<head>
</head>
<body>
<div id="test">
<h2>background should be green</h2>
</div>
<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.7.2/jquery.min.js"></script>
<script>
var green = 'data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVQImWNgaGD4DwAChAGA2FJdiQAAAABJRU5ErkJggg==';
var red = 'data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVQImWP4z8DwHwAFAAH/q842iQAAAABJRU5ErkJggg==';
$(function() {
var $t = $("#test");
$t.css('background-image', 'url("' + green + '")');
$t.css('background-image', 'url("' + red + '"'); // ) is missing
})
</script>
</body>
</html>
Actual results:
the second invalid attempt to set the background image is accepted too (IE9 and Chrome 21 both reject it)
Expected results:
background green
should have thrown syntax error ? (not sure i would want that tough)
Attachment #653837 -
Attachment mime type: text/plain → text/html
|
||
Comment 2•12 years ago
|
||
The spec requires this behavior. Yes, IE and Chrome get it wrong.
Specifically, see http://www.w3.org/TR/CSS21/syndata.html#parsing-errors under "Unexpected end of style sheet".
Status: UNCONFIRMED → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
|
|
||
Comment 3•12 years ago
|
||
And note that you don't even need the dynamic stuff. Just doing:
<style>
body { background-image: url("something.png"</style>
will trigger the relevant error-handling codepath in UAs that implement the spec properly.
You need to log in
before you can comment on or make changes to this bug.
Description
•