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

Unified Diff: LayoutTests/fast/table/script-tests/border-changes.js

Issue 789513004: Removing border attribute should remove the visual border (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Patch for landing Created 6 years 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/table/script-tests/border-changes.js
diff --git a/LayoutTests/fast/table/script-tests/border-changes.js b/LayoutTests/fast/table/script-tests/border-changes.js
index e532931f6056367ba3f65909403819d99ce09f9f..87ccb15cd0325da459944f4cb64cbcffb7b8a130 100644
--- a/LayoutTests/fast/table/script-tests/border-changes.js
+++ b/LayoutTests/fast/table/script-tests/border-changes.js
@@ -153,4 +153,20 @@ shouldBe("getComputedStyle(cell, '').borderRightColor", "red");
shouldBe("getComputedStyle(cell, '').borderTopColor", "red");
shouldBe("getComputedStyle(cell, '').borderBottomColor", "red");
+// resets to default border width of 3px
+table.removeAttribute("border");
+
+shouldBe("getComputedStyle(table, '').borderLeftWidth", "'3px'");
+shouldBe("getComputedStyle(table, '').borderRightWidth", "'3px'");
+shouldBe("getComputedStyle(table, '').borderTopWidth", "'3px'");
+shouldBe("getComputedStyle(table, '').borderBottomWidth", "'3px'");
+shouldBe("getComputedStyle(table, '').borderLeftStyle", "'solid'");
+shouldBe("getComputedStyle(table, '').borderRightStyle", "'solid'");
+shouldBe("getComputedStyle(table, '').borderTopStyle", "'solid'");
+shouldBe("getComputedStyle(table, '').borderBottomStyle", "'solid'");
+shouldBe("getComputedStyle(table, '').borderLeftColor", "yellow");
+shouldBe("getComputedStyle(table, '').borderRightColor", "yellow");
+shouldBe("getComputedStyle(table, '').borderTopColor", "yellow");
+shouldBe("getComputedStyle(table, '').borderBottomColor", "yellow");
+
document.body.removeChild(table);
« no previous file with comments | « LayoutTests/fast/table/border-remove-border-attribute-expected.html ('k') | Source/core/html/HTMLElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698