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

Unified Diff: third_party/WebKit/LayoutTests/fast/css/getPropertyValue-border.html

Issue 1982903002: Fix 'border' serialization to fail in cssText when it is invalid (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/css/getPropertyValue-border.html
diff --git a/third_party/WebKit/LayoutTests/fast/css/getPropertyValue-border.html b/third_party/WebKit/LayoutTests/fast/css/getPropertyValue-border.html
index d8751e65cc0b9d9767e95fea4a9985083ee9b87e..11fef2ad5bd691f0603a534fac1fd2e6e8cdc379 100644
--- a/third_party/WebKit/LayoutTests/fast/css/getPropertyValue-border.html
+++ b/third_party/WebKit/LayoutTests/fast/css/getPropertyValue-border.html
@@ -33,27 +33,25 @@
description("<a href=\"https://bugs.webkit.org/show_bug.cgi?id=15823\">Bug 15823: getPropertyValue for border returns null, should compute the shorthand value</a>");
var div1 = document.getElementById("border1");
- shouldBe('div1.style.getPropertyValue("border")', "'5px solid green'");
+ shouldBeEqualToString('div1.style.getPropertyValue("border")', "5px solid green");
var div2 = document.getElementById("border2");
- shouldBe('div2.style.getPropertyValue("border")', "'5px solid'");
+ shouldBeEqualToString('div2.style.getPropertyValue("border")', "");
var div3 = document.getElementById("border3");
- shouldBe('div3.style.getPropertyValue("border")', "'5px green'");
- debug(" NOTE: '5px green' is an illegal CSS value for 'border'.");
+ shouldBeEqualToString('div3.style.getPropertyValue("border")', "");
var div4 = document.getElementById("border4");
- shouldBe('div4.style.getPropertyValue("border")', "'solid green'");
+ shouldBeEqualToString('div4.style.getPropertyValue("border")', "");
var div5 = document.getElementById("border5");
- shouldBe('div5.style.getPropertyValue("border")', "'green'");
- debug(" NOTE: 'green' is an illegal CSS value for 'border'.");
+ shouldBeEqualToString('div5.style.getPropertyValue("border")', "");
var div6 = document.getElementById("border6");
- shouldBe('div6.style.getPropertyValue("border")', "'5px'");
+ shouldBeEqualToString('div6.style.getPropertyValue("border")', "");
var div7 = document.getElementById("border7");
- shouldBe('div7.style.getPropertyValue("border")', "'solid'");
+ shouldBeEqualToString('div7.style.getPropertyValue("border")', "");
var div8 = document.getElementById("border8");
shouldBeEqualToString('div8.style.getPropertyValue("border")', "");

Powered by Google App Engine
This is Rietveld 408576698