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

Unified Diff: LayoutTests/fast/backgrounds/size/resources/parsing-background-size-values.js

Issue 764703002: Support unit-less lengths for white-list of properties only. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: min-width/height still accepts unitless lengths Created 5 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fast/backgrounds/size/resources/parsing-background-size-values.js
diff --git a/LayoutTests/fast/backgrounds/size/resources/parsing-background-size-values.js b/LayoutTests/fast/backgrounds/size/resources/parsing-background-size-values.js
index df9ba63b0c39e7d17c0469189b06e7928fcd7516..3a4021b8d6607938171d0eaf51f736d30b49815b 100644
--- a/LayoutTests/fast/backgrounds/size/resources/parsing-background-size-values.js
+++ b/LayoutTests/fast/backgrounds/size/resources/parsing-background-size-values.js
@@ -13,7 +13,7 @@ function test(value)
shouldBe('test("background-size: contain;")', '"contain"');
shouldBe('test("background-size: cover;")', '"cover"');
-shouldBe('test("background-size: 100 100;")', '"100px 100px"');
+shouldBe('test("background-size: 100 100;")', 'null');
shouldBe('test("background-size: 100px 100px;")', '"100px 100px"');
shouldBe('test("background-size: auto 50px;")', '"auto 50px"');
shouldBe('test("background-size: 50px auto;")', '"50px"');
@@ -23,7 +23,7 @@ shouldBe('test("background-size: 4em auto;")', '"4em"');
shouldBe('test("background-size: 5em;")', '"5em"');
shouldBe('test("-webkit-background-size: 5em ;")', '"5em"');
-shouldBe('test("background-size: 100 100 100;")', 'null');
+shouldBe('test("background-size: 100px 100px 100px;")', 'null');
shouldBe('test("background-size: coconut;")', 'null');
shouldBe('test("background-size: 100px,;")', 'null');

Powered by Google App Engine
This is Rietveld 408576698