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

Unified Diff: LayoutTests/fast/css/font-shorthand-from-longhands.html

Issue 203523002: Reland "Add plumbing for font-stretch" (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: w/fix for ASSERT Created 6 years, 5 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/css/font-shorthand-from-longhands.html
diff --git a/LayoutTests/fast/css/font-shorthand-from-longhands.html b/LayoutTests/fast/css/font-shorthand-from-longhands.html
index 1df1c2fb5bd03e73462d3923bbad3204ebef6828..c8544fcf49113fdbc7369fce4e4787548980d580 100644
--- a/LayoutTests/fast/css/font-shorthand-from-longhands.html
+++ b/LayoutTests/fast/css/font-shorthand-from-longhands.html
@@ -36,50 +36,50 @@ function checkFontStyleValue() {
style.fontSize = "20px";
// We need at least the font-family to build the shorthand.
shouldBe("style.font", "''");
-shouldBe("computedStyle.font", "'normal normal normal 20px/normal foobar'");
+shouldBe("computedStyle.font", "'normal normal normal normal 20px/normal foobar'");
shouldBe("computedStyle.fontSize", "'20px'");
shouldBe("checkFontStyleValue()", "true");
style.fontSize = "20px";
style.fontFamily = "sans-serif";
shouldBe("style.font", "'20px sans-serif'");
-shouldBe("computedStyle.font", "'normal normal normal 20px/normal sans-serif'");
+shouldBe("computedStyle.font", "'normal normal normal normal 20px/normal sans-serif'");
shouldBe("computedStyle.fontFamily", "'sans-serif'");
shouldBe("checkFontStyleValue()", "true");
style.fontStyle = "italic";
shouldBe("style.font", "'italic 20px sans-serif'");
-shouldBe("computedStyle.font", "'italic normal normal 20px/normal sans-serif'");
+shouldBe("computedStyle.font", "'italic normal normal normal 20px/normal sans-serif'");
shouldBe("computedStyle.fontStyle", "'italic'");
shouldBe("checkFontStyleValue()", "true");
style.fontVariant = "small-caps";
shouldBe("style.font", "'italic small-caps 20px sans-serif'");
-shouldBe("computedStyle.font", "'italic small-caps normal 20px/normal sans-serif'");
+shouldBe("computedStyle.font", "'italic small-caps normal normal 20px/normal sans-serif'");
shouldBe("computedStyle.fontVariant", "'small-caps'");
shouldBe("checkFontStyleValue()", "true");
style.fontWeight = "bold";
shouldBe("style.font", "'italic small-caps bold 20px sans-serif'");
-shouldBe("computedStyle.font", "'italic small-caps bold 20px/normal sans-serif'");
+shouldBe("computedStyle.font", "'italic small-caps bold normal 20px/normal sans-serif'");
shouldBe("computedStyle.fontWeight", "'bold'");
shouldBe("checkFontStyleValue()", "true");
style.lineHeight = "40px";
shouldBe("style.font", "'italic small-caps bold 20px/40px sans-serif'");
-shouldBe("computedStyle.font", "'italic small-caps bold 20px/40px sans-serif'");
+shouldBe("computedStyle.font", "'italic small-caps bold normal 20px/40px sans-serif'");
shouldBe("computedStyle.lineHeight", "'40px'");
shouldBe("checkFontStyleValue()", "true");
style.font = "";
shouldBe("style.font", "''");
-shouldBe("computedStyle.font", "'normal normal normal 16px/normal foobar'");
+shouldBe("computedStyle.font", "'normal normal normal normal 16px/normal foobar'");
shouldBe("checkFontStyleValue()", "true");
style.fontWeight = "bold";
// It is normal to return null as the font-size is mandatory to build the shorthand.
shouldBe("style.font", "''");
-shouldBe("computedStyle.font", "'normal normal bold 16px/normal foobar'");
+shouldBe("computedStyle.font", "'normal normal bold normal 16px/normal foobar'");
shouldBe("computedStyle.fontWeight", "'bold'");
shouldBe("checkFontStyleValue()", "true");
@@ -87,7 +87,7 @@ style.fontSize = "40px";
style.fontFamily = "sans-serif";
style.fontWeight = "bold";
shouldBe("style.font", "'bold 40px sans-serif'");
-shouldBe("computedStyle.font", "'normal normal bold 40px/normal sans-serif'");
+shouldBe("computedStyle.font", "'normal normal bold normal 40px/normal sans-serif'");
shouldBe("computedStyle.fontSize", "'40px'");
shouldBe("computedStyle.fontFamily", "'sans-serif'");
shouldBe("checkFontStyleValue()", "true");
« no previous file with comments | « LayoutTests/fast/css/font-shorthand-expected.txt ('k') | LayoutTests/fast/css/font-shorthand-from-longhands-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698