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)

x86_64
Linux
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: hsivonen, Assigned: hsivonen)

Details

Attachments

(2 files)

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)"
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: nobody → hsivonen
Status: NEW → ASSIGNED
Attached patch Parser fixSplinter Review
Attached patch Test case fixSplinter Review
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)
Summary: [HTML5][Spec] layout/style/test/test_parse_rule.html fails → [HTML5][Patch] layout/style/test/test_parse_rule.html fails
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.

Attachment

General

Created:
Updated:
Size: