Closed
Bug 546641
Opened 14 years ago
Closed 14 years ago
[HTML5][Patch] layout/style/test/test_parse_rule.html fails
Categories
(Core :: DOM: HTML Parser, defect, P1)
Tracking
()
RESOLVED
FIXED
People
(Reporter: hsivonen, Assigned: hsivonen)
Details
Attachments
(2 files)
1.95 KB,
patch
|
Details | Diff | Splinter Review | |
1.16 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
80036 ERROR TEST-UNEXPECTED-FAIL | /tests/layout/style/test/test_parse_rule.html | div[title='zxcv weeqweqeweasd\D\A a']{color:green} - got "rgb(0, 0, 0)", expected "rgb(0, 128, 0)"
Assignee | ||
Comment 1•14 years ago
|
||
Simplified test case: http://hsivonen.iki.fi/test/moz/escaped-crlf.html
Assignee | ||
Comment 2•14 years ago
|
||
This is a spec feature: is defined to expand to LF.
Summary: [HTML5] layout/style/test/test_parse_rule.html fails → [HTML5][Spec] layout/style/test/test_parse_rule.html fails
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•14 years ago
|
||
Assignee | ||
Comment 4•14 years ago
|
||
The parser fix make expand to CR. See http://www.w3.org/Bugs/Public/show_bug.cgi?id=9144 The test case fix removes a trailing from the test attribute, because the HTML5 parsing algorithm doesn't trim trailing white space from attribute values (a simplification that matches IE) but the test selectors assume the trailing WS doesn't end up in the attribute value.
Attachment #431057 -
Flags: review?(roc)
Assignee | ||
Updated•14 years ago
|
Summary: [HTML5][Spec] layout/style/test/test_parse_rule.html fails → [HTML5][Patch] layout/style/test/test_parse_rule.html fails
Attachment #431057 -
Flags: review?(roc) → review+
Assignee | ||
Comment 5•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/86215ba5e99e (code) http://hg.mozilla.org/mozilla-central/rev/728f6d5714cd (test)
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•