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

Unified Diff: LayoutTests/fast/css/script-tests/image-set-parsing-invalid.js

Issue 714693002: Don't require getPropertyCSSValue in fast/css tests (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 1 month 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/css/script-tests/image-set-parsing-invalid.js
diff --git a/LayoutTests/fast/css/script-tests/image-set-parsing-invalid.js b/LayoutTests/fast/css/script-tests/image-set-parsing-invalid.js
index a84205e9e963d1125791acd141e9aa58af30cd93..8c924049e47138505d444aac7f4fdd9c485cb11a 100644
--- a/LayoutTests/fast/css/script-tests/image-set-parsing-invalid.js
+++ b/LayoutTests/fast/css/script-tests/image-set-parsing-invalid.js
@@ -3,26 +3,26 @@ description("Test the parsing of the -webkit-image-set function.");
// These have to be global for the test helpers to see them.
var cssRule;
-function testInvalidImageSet(description, property, rule)
+function testInvalidImageSet(description, rule)
{
debug("");
debug(description + " : " + rule);
var div = document.createElement("div");
- div.setAttribute("style", property + ": -webkit-image-set(" + rule + ")");
+ div.style.backgroundImage = "-webkit-image-set(" + rule + ")";
document.body.appendChild(div);
- cssRule = div.style.getPropertyCSSValue(property);
- shouldBe("cssRule", "null");
+ cssRule = div.style.backgroundImage;
+ shouldBeEmptyString("cssRule");
document.body.removeChild(div);
}
-testInvalidImageSet("Too many url parameters", "background-image", "url(#a #b)");
-testInvalidImageSet("No x", "background-image", "url('#a') 1");
-testInvalidImageSet("No comma", "background-image", "url('#a') 1x url('#b') 2x");
-testInvalidImageSet("Too many scale factor parameters", "background-image", "url('#a') 1x 2x");
-testInvalidImageSet("Scale factor is 0", "background-image", "url('#a') 0x");
-testInvalidImageSet("No url function", "background-image", "'#a' 1x");
+testInvalidImageSet("Too many url parameters", "url(#a #b)");
+testInvalidImageSet("No x", "url('#a') 1");
+testInvalidImageSet("No comma", "url('#a') 1x url('#b') 2x");
+testInvalidImageSet("Too many scale factor parameters", "url('#a') 1x 2x");
+testInvalidImageSet("Scale factor is 0", "url('#a') 0x");
+testInvalidImageSet("No url function", "'#a' 1x");
successfullyParsed = true;
« no previous file with comments | « LayoutTests/fast/css/script-tests/image-set-parsing.js ('k') | LayoutTests/fast/css/sheet-collection-link.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698