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

Unified Diff: third_party/WebKit/LayoutTests/fast/css/font-shorthand-line-height.html

Issue 1820773002: getComputedStyle() should return used value for 'line-height' (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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/font-shorthand-line-height.html
diff --git a/third_party/WebKit/LayoutTests/fast/css/font-shorthand-line-height.html b/third_party/WebKit/LayoutTests/fast/css/font-shorthand-line-height.html
index 0e4160963101e91dd9e850d70227b400b6441edb..a912dc99c8b858e40645b4a5cc8b0105db589304 100644
--- a/third_party/WebKit/LayoutTests/fast/css/font-shorthand-line-height.html
+++ b/third_party/WebKit/LayoutTests/fast/css/font-shorthand-line-height.html
@@ -24,9 +24,9 @@
}
shouldBe("lineHeight(system_font_1)", "'100px'");
- shouldBe("lineHeight(system_font_2)", "'normal'");
+ shouldNotBe("lineHeight(system_font_2)", "'100px'");
shouldBe("lineHeight(shorthand_normal_1)", "'100px'");
- shouldBe("lineHeight(shorthand_normal_2)", "'normal'");
+ shouldNotBe("lineHeight(shorthand_normal_2)", "'100px'");
shouldBe("lineHeight(shorthand_line_height_1)", "'100px'");
shouldBe("lineHeight(shorthand_line_height_2)", "'200px'");
</script>

Powered by Google App Engine
This is Rietveld 408576698