LayoutTests/fast/css3-text/css3-text-decoration/getComputedStyle/script-tests/getComputedStyle-text-underline-position.js - Issue 57493002: Remove 'alphabetic' value from text-underline-position

Unified Diff: LayoutTests/fast/css3-text/css3-text-decoration/getComputedStyle/script-tests/getComputedStyle-text-underline-position.js

Issue 57493002: Remove 'alphabetic' value from text-underline-position (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Patch for landing Created 7 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

« no previous file with comments | « LayoutTests/fast/css3-text/css3-text-decoration/getComputedStyle/getComputedStyle-text-underline-position-expected.txt ('k') | LayoutTests/fast/css3-text/css3-text-decoration/text-underline-position/style.css » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/fast/css3-text/css3-text-decoration/getComputedStyle/script-tests/getComputedStyle-text-underline-position.js

diff --git a/LayoutTests/fast/css3-text/css3-text-decoration/getComputedStyle/script-tests/getComputedStyle-text-underline-position.js b/LayoutTests/fast/css3-text/css3-text-decoration/getComputedStyle/script-tests/getComputedStyle-text-underline-position.js

index a18322e560ceb0cbb6b05a6c1270218a3307405a..73363eaa60228839c15d8cc162e15c0d37b68064 100644

--- a/LayoutTests/fast/css3-text/css3-text-decoration/getComputedStyle/script-tests/getComputedStyle-text-underline-position.js

+++ b/LayoutTests/fast/css3-text/css3-text-decoration/getComputedStyle/script-tests/getComputedStyle-text-underline-position.js

@@ -18,7 +18,7 @@ function testComputedStyle(propertyJS, propertyCSS, type, value)

description("Test to make sure text-underline-position property returns values properly.")

-// FIXME: This test tests property values 'auto', 'alphabetic' and 'under'. We don't fully match

+// FIXME: This test tests property values 'auto' and 'under'. We don't fully match

// the specification as we don't support [ left | right ] and this is left for another implementation

// as the rendering will need to be added.

@@ -52,12 +52,6 @@ testElementStyle("textUnderlinePosition", "text-underline-position", "[object CS

testComputedStyle("textUnderlinePosition", "text-underline-position", "[object CSSPrimitiveValue]", "auto");

debug('');

-debug("Value 'alphabetic':");

-e.style.textUnderlinePosition = 'alphabetic';

-testElementStyle("textUnderlinePosition", "text-underline-position", "[object CSSPrimitiveValue]", "alphabetic");

-testComputedStyle("textUnderlinePosition", "text-underline-position", "[object CSSPrimitiveValue]", "alphabetic");

-debug('');

debug("Value 'under':");

e.style.textUnderlinePosition = 'under';

testElementStyle("textUnderlinePosition", "text-underline-position", "[object CSSPrimitiveValue]", "under");

@@ -71,11 +65,6 @@ testElementStyle("textUnderlinePosition", "text-underline-position", null, "");

testComputedStyle("textUnderlinePosition", "text-underline-position", "[object CSSPrimitiveValue]", "under");

debug('');

-debug("Value 'auto alphabetic':");

-e.style.textUnderlinePosition = 'auto alphabetic';

-testElementStyle("textUnderlinePosition", "text-underline-position", null, "");

-debug('');

debug("Value 'auto under':");

e.style.textUnderlinePosition = 'auto under';

testElementStyle("textUnderlinePosition", "text-underline-position", null, "");