Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(496)

Side by Side Diff: LayoutTests/fast/css/script-tests/invalid-predefined-color.js

Issue 1151593003: Remove hexDigit check in CSSParserValueList (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Remove ASSERT and add subtest Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 description( 1 description(
2 "This test checks hexadecimal color parsing." 2 "This test checks hexadecimal color parsing."
3 ); 3 );
4 4
5 var red = "rgb(255, 0, 0)"; 5 var red = "rgb(255, 0, 0)";
6 var black = "rgb(0, 0, 0)"; 6 var black = "rgb(0, 0, 0)";
7 7
8 for (var i=1;i<=5;i++) 8 for (var i=1;i<=5;i++)
9 { 9 {
10 var div = document.getElementById("valid"+i); 10 var div = document.getElementById("valid"+i);
11 shouldBe("getComputedStyle(div, null).color", "red"); 11 shouldBe("getComputedStyle(div, null).color", "red");
12 } 12 }
13 13
14 for (var i=1;i<=5;i++) 14 for (var i=1;i<=6;i++)
15 { 15 {
16 var div = document.getElementById("invalid"+i); 16 var div = document.getElementById("invalid"+i);
17 shouldBe("getComputedStyle(div, null).color", "black"); 17 shouldBe("getComputedStyle(div, null).color", "black");
18 } 18 }
OLDNEW
« no previous file with comments | « LayoutTests/fast/css/invalid-predefined-color-expected.txt ('k') | Source/core/css/parser/CSSParserValues.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698