Remove -webkit-marquee.
authoremilio@crisal.io <emilio@crisal.io@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 15 Dec 2019 20:19:07 +0000 (20:19 +0000)
committeremilio@crisal.io <emilio@crisal.io@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 15 Dec 2019 20:19:07 +0000 (20:19 +0000)
https://bugs.webkit.org/show_bug.cgi?id=117769

Reviewed by Simon Fraser.

Source/WebCore:

This doesn't simplify the code so much yet but makes the CSS properties not
accessible by web content, which means that how marquee is implemented is now an
implementation detail that WebKit can change.

Had to keep some of parsing code because addHTMLLengthToStyle uses the
CSS parser, which is a bit unfortunate. But we can avoid dealing with
identifiers so it can be simplified a bit, and similarly we can avoid
having a shorthand altogether.

Covered by existing tests that are being modified to reflect the
change.

* css/CSSComputedStyleDeclaration.cpp:
(WebCore::ComputedStyleExtractor::valueForPropertyInStyle):
* css/CSSProperties.json:
* css/StyleProperties.cpp:
(WebCore::StyleProperties::getPropertyValue const):
* css/parser/CSSPropertyParser.cpp:
(WebCore::CSSPropertyParser::parseSingleValue):
(WebCore::CSSPropertyParser::parseShorthand):

Source/WebInspectorUI:

* UserInterface/Models/CSSKeywordCompletions.js:

LayoutTests:

* fast/css/getComputedStyle/computed-style-expected.txt:
* fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* fast/css/getComputedStyle/getComputedStyle-length-unit-expected.txt:
* fast/css/getComputedStyle/getComputedStyle-length-unit.html:
* fast/css/getComputedStyle/getComputedStyle-zoom-and-background-size-expected.txt:
* fast/css/getComputedStyle/getComputedStyle-zoom-and-background-size.html:
* fast/css/getComputedStyle/resources/property-names.js:
* fast/css/getPropertyValue-webkit-marquee-expected.txt: Removed.
* fast/css/getPropertyValue-webkit-marquee.html: Removed.
* fast/css/inherit-initial-shorthand-values-expected.txt:
* fast/css/inherit-initial-shorthand-values.html:
* fast/css/remove-shorthand-expected.txt:
* fast/css/remove-shorthand.html:
* legacy-animation-engine/fast/css/getComputedStyle/resources/property-names.js:
* platform/gtk/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/gtk/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/gtk/svg/css/getComputedStyle-basic-expected.txt:
* platform/ios/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/ios/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/ios/svg/css/getComputedStyle-basic-expected.txt:
* platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/mac/svg/css/getComputedStyle-basic-expected.txt:
* platform/wpe/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/wpe/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/wpe/svg/css/getComputedStyle-basic-expected.txt:
* svg/css/getComputedStyle-basic-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@253540 268f45cc-cd09-0410-ab3c-d52691b4dbfc

36 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/fast/css/getComputedStyle/getComputedStyle-length-unit-expected.txt
LayoutTests/fast/css/getComputedStyle/getComputedStyle-length-unit.html
LayoutTests/fast/css/getComputedStyle/getComputedStyle-zoom-and-background-size-expected.txt
LayoutTests/fast/css/getComputedStyle/getComputedStyle-zoom-and-background-size.html
LayoutTests/fast/css/getComputedStyle/resources/property-names.js
LayoutTests/fast/css/getPropertyValue-webkit-marquee-expected.txt [deleted file]
LayoutTests/fast/css/getPropertyValue-webkit-marquee.html [deleted file]
LayoutTests/fast/css/inherit-initial-shorthand-values-expected.txt
LayoutTests/fast/css/inherit-initial-shorthand-values.html
LayoutTests/fast/css/remove-shorthand-expected.txt
LayoutTests/fast/css/remove-shorthand.html
LayoutTests/legacy-animation-engine/fast/css/getComputedStyle/resources/property-names.js
LayoutTests/platform/gtk/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/platform/gtk/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/gtk/svg/css/getComputedStyle-basic-expected.txt
LayoutTests/platform/ios/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/platform/ios/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/ios/svg/css/getComputedStyle-basic-expected.txt
LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/mac/svg/css/getComputedStyle-basic-expected.txt
LayoutTests/platform/wpe/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/platform/wpe/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/wpe/svg/css/getComputedStyle-basic-expected.txt
LayoutTests/svg/css/getComputedStyle-basic-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/css/CSSComputedStyleDeclaration.cpp
Source/WebCore/css/CSSProperties.json
Source/WebCore/css/StyleProperties.cpp
Source/WebCore/css/parser/CSSPropertyParser.cpp
Source/WebCore/style/StyleBuilderConverter.h
Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Models/CSSKeywordCompletions.js

index ebe88cc..bc16a2f 100644 (file)
@@ -1,3 +1,38 @@
+2019-12-15  Emilio Cobos Álvarez  <emilio@crisal.io>
+
+        Remove -webkit-marquee.
+        https://bugs.webkit.org/show_bug.cgi?id=117769
+
+        Reviewed by Simon Fraser.
+
+        * fast/css/getComputedStyle/computed-style-expected.txt:
+        * fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * fast/css/getComputedStyle/getComputedStyle-length-unit-expected.txt:
+        * fast/css/getComputedStyle/getComputedStyle-length-unit.html:
+        * fast/css/getComputedStyle/getComputedStyle-zoom-and-background-size-expected.txt:
+        * fast/css/getComputedStyle/getComputedStyle-zoom-and-background-size.html:
+        * fast/css/getComputedStyle/resources/property-names.js:
+        * fast/css/getPropertyValue-webkit-marquee-expected.txt: Removed.
+        * fast/css/getPropertyValue-webkit-marquee.html: Removed.
+        * fast/css/inherit-initial-shorthand-values-expected.txt:
+        * fast/css/inherit-initial-shorthand-values.html:
+        * fast/css/remove-shorthand-expected.txt:
+        * fast/css/remove-shorthand.html:
+        * legacy-animation-engine/fast/css/getComputedStyle/resources/property-names.js:
+        * platform/gtk/fast/css/getComputedStyle/computed-style-expected.txt:
+        * platform/gtk/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/gtk/svg/css/getComputedStyle-basic-expected.txt:
+        * platform/ios/fast/css/getComputedStyle/computed-style-expected.txt:
+        * platform/ios/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/ios/svg/css/getComputedStyle-basic-expected.txt:
+        * platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
+        * platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/mac/svg/css/getComputedStyle-basic-expected.txt:
+        * platform/wpe/fast/css/getComputedStyle/computed-style-expected.txt:
+        * platform/wpe/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/wpe/svg/css/getComputedStyle-basic-expected.txt:
+        * svg/css/getComputedStyle-basic-expected.txt:
+
 2019-12-14  Per Arne Vollan  <pvollan@apple.com>
 
         [iOS] Deny mach lookup access to "*.viewservice" in the WebContent process
index e92f538..6f36e59 100644 (file)
@@ -233,10 +233,6 @@ zoom: 1;
 -webkit-locale: auto;
 -webkit-margin-after-collapse: collapse;
 -webkit-margin-before-collapse: collapse;
--webkit-marquee-direction: auto;
--webkit-marquee-increment: 6px;
--webkit-marquee-repetition: infinite;
--webkit-marquee-style: scroll;
 -webkit-mask-box-image: none;
 -webkit-mask-box-image-outset: 0px;
 -webkit-mask-box-image-repeat: stretch;
index 9a9ca47..be6f9f3 100644 (file)
@@ -232,10 +232,6 @@ zoom: 1
 -webkit-locale: auto
 -webkit-margin-after-collapse: collapse
 -webkit-margin-before-collapse: collapse
--webkit-marquee-direction: auto
--webkit-marquee-increment: 6px
--webkit-marquee-repetition: infinite
--webkit-marquee-style: scroll
 -webkit-mask-box-image: none
 -webkit-mask-box-image-outset: 0px
 -webkit-mask-box-image-repeat: stretch
index 2dac7e3..c386e94 100644 (file)
@@ -14,7 +14,6 @@ PASS testPropertyUnit('visible', 'outline-width') is 'px'
 PASS testPropertyUnit('visible', '-webkit-column-rule-width') is 'px'
 PASS testPropertyUnit('visible', '-webkit-column-width') is 'px'
 PASS testPropertyUnit('visible', '-webkit-column-gap') is 'px'
-PASS testPropertyUnit('visible', '-webkit-marquee-increment') is 'px'
 PASS testPropertyUnit('visible', '-webkit-mask-position') is 'px'
 PASS testPropertyUnit('visible', '-webkit-mask-size') is 'px'
 PASS testPropertyUnit('visible', '-webkit-perspective') is 'px'
@@ -59,7 +58,6 @@ PASS testPropertyUnit('hidden', 'outline-width') is 'px'
 PASS testPropertyUnit('hidden', '-webkit-column-rule-width') is 'px'
 PASS testPropertyUnit('hidden', '-webkit-column-width') is 'px'
 PASS testPropertyUnit('hidden', '-webkit-column-gap') is 'px'
-PASS testPropertyUnit('hidden', '-webkit-marquee-increment') is 'px'
 PASS testPropertyUnit('hidden', '-webkit-mask-position') is 'px'
 PASS testPropertyUnit('hidden', '-webkit-mask-size') is 'px'
 PASS testPropertyUnit('hidden', '-webkit-perspective') is 'px'
index a5a5557..79efe78 100644 (file)
@@ -31,7 +31,6 @@
             "-webkit-column-width": "80px",
             "-webkit-column-gap": "20px",
 
-            "-webkit-marquee-increment": "10px",
             "-webkit-mask-position": "10px 10px",
             "-webkit-mask-size": "10px 10px",
             "-webkit-perspective": "400px",
index d703b98..2da5826 100644 (file)
@@ -24,7 +24,6 @@ outline-width PASS    2px     2px
 -webkit-column-rule-style      PASS    solid   solid
 -webkit-column-width   PASS    80px    80px
 -webkit-column-gap     PASS    20px    20px
--webkit-marquee-increment      PASS    10px    10px
 -webkit-mask-position  PASS    10px 10px       10px 10px
 -webkit-mask-size      PASS    10px 10px       10px 10px
 -webkit-perspective    PASS    400px   400px
@@ -77,7 +76,6 @@ outline-width PASS    2px     2px
 -webkit-column-rule-style      PASS    solid   solid
 -webkit-column-width   PASS    80px    80px
 -webkit-column-gap     PASS    20px    20px
--webkit-marquee-increment      PASS    10px    10px
 -webkit-mask-position  PASS    10px 10px       10px 10px
 -webkit-mask-size      PASS    10px 10px       10px 10px
 -webkit-perspective    PASS    400px   400px
index d3298d4..195fc9e 100644 (file)
       "-webkit-column-width": "80px",
       "-webkit-column-gap": "20px",
       
-      "-webkit-marquee-increment": "10px",
       "-webkit-mask-position": "10px 10px",
       "-webkit-mask-size": "10px 10px",
       "-webkit-perspective": "400px",
 
 
 </body>
-</html>
\ No newline at end of file
+</html>
index 2b5070f..29ff284 100644 (file)
@@ -81,10 +81,6 @@ var propertiesToTest = {
     "-webkit-locale": true,
     "-webkit-margin-after-collapse": true,
     "-webkit-margin-before-collapse": true,
-    "-webkit-marquee-direction": true,
-    "-webkit-marquee-increment": true,
-    "-webkit-marquee-repetition": true,
-    "-webkit-marquee-style": true,
     "-webkit-mask-attachment": true,
     "-webkit-mask-box-image": true,
     "-webkit-mask-box-image-outset": true,
diff --git a/LayoutTests/fast/css/getPropertyValue-webkit-marquee-expected.txt b/LayoutTests/fast/css/getPropertyValue-webkit-marquee-expected.txt
deleted file mode 100644 (file)
index ba6c3e3..0000000
+++ /dev/null
@@ -1,25 +0,0 @@
-Bug 110816: getPropertyValue for -webkit-marquee returns null, should compute the shorthand value
-
-On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
-
-
-PASS webkitMarqueeValue("marquee1") is 'forwards medium 10 slide normal'
-PASS webkitMarqueeValue("marquee2") is 'auto large 10 slide'
-PASS webkitMarqueeValue("marquee3") is 'forwards infinite slide normal'
-PASS webkitMarqueeValue("marquee4") is 'forwards medium slide normal'
-PASS webkitMarqueeValue("marquee5") is 'medium 10 slide normal'
-PASS webkitMarqueeValue("marquee6") is 'forwards small normal'
-PASS webkitMarqueeValue("marquee7") is 'forwards medium infinite'
-PASS webkitMarqueeValue("marquee8") is 'slide fast'
-PASS webkitMarqueeValue("marquee9") is 'forwards'
-PASS webkitMarqueeValue("marquee10") is 'alternate'
-PASS webkitMarqueeValue("marquee11") is 'infinite'
-NOTE: 'foo' is an illegal CSS value for '-webkit-marquee'.
-PASS webkitMarqueeValue("marquee12") is ""
-NOTE: If only few longhand properties are specified, getPropertyValue for shorthand property returns null.
-PASS webkitMarqueeValue("marquee13") is ""
-PASS webkitMarqueeValue("marquee14") is ""
-PASS successfullyParsed is true
-
-TEST COMPLETE
-            
diff --git a/LayoutTests/fast/css/getPropertyValue-webkit-marquee.html b/LayoutTests/fast/css/getPropertyValue-webkit-marquee.html
deleted file mode 100644 (file)
index c4d5cb3..0000000
+++ /dev/null
@@ -1,61 +0,0 @@
-<!DOCTYPE>
-<html>
-  <head>
-    <title>-webkit-marquee shorthand getPropertyValue test</title>
-    <style type="text/css">
-      .test {
-        display: inline-block;
-        height: 5em;
-        width: 10em;
-      }
-    </style>
-    <script src="../../resources/js-test-pre.js"></script>
-    <script>
-      if (window.testRunner)
-        testRunner.dumpAsText();
-    </script>
-  </head>
-  <body>
-    <marquee id="marquee1" class="test" style="-webkit-marquee: forwards medium 10 slide normal;"></<marquee>
-    <marquee id="marquee2" class="test" style="-webkit-marquee: auto large 10 slide;"></marquee>
-    <marquee id="marquee3" class="test" style="-webkit-marquee: forwards infinite slide normal;"></marquee>
-    <marquee id="marquee4" class="test" style="-webkit-marquee: forwards medium slide normal;"></marquee>
-    <marquee id="marquee5" class="test" style="-webkit-marquee: medium 10 slide normal;"></marquee>
-    <marquee id="marquee6" class="test" style="-webkit-marquee: forwards small normal;"></marquee>
-    <marquee id="marquee7" class="test" style="-webkit-marquee: forwards medium infinite;"></marquee>
-    <marquee id="marquee8" class="test" style="-webkit-marquee: slide fast;"></marquee>
-    <marquee id="marquee9" class="test" style="-webkit-marquee: forwards;"></marquee>
-    <marquee id="marquee10" class="test" style="-webkit-marquee: alternate;"></marquee>
-    <marquee id="marquee11" class="test" style="-webkit-marquee: infinite;"></marquee>
-    <marquee id="marquee12" class="test" style="-webkit-marquee: foo;"></marquee>
-    <marquee id="marquee13" class="test" style="-webkit-marquee-direction: forwards; -webkit-marquee-speed: fast;"></marquee>
-    <marquee id="marquee14" class="test" style="-webkit-marquee-repetition: infinite; -webkit-marquee-style: none;"></marquee>
-
-    <script>
-      description("<a href=\"https://bugs.webkit.org/show_bug.cgi?id=110816\">Bug 110816: getPropertyValue for -webkit-marquee returns null, should compute the shorthand value</a>");
-
-      function webkitMarqueeValue(id) {
-        var element = document.getElementById(id);
-        return element.style.getPropertyValue("-webkit-marquee");
-      }
-
-      shouldBe('webkitMarqueeValue("marquee1")', "'forwards medium 10 slide normal'");
-      shouldBe('webkitMarqueeValue("marquee2")', "'auto large 10 slide'");
-      shouldBe('webkitMarqueeValue("marquee3")', "'forwards infinite slide normal'");
-      shouldBe('webkitMarqueeValue("marquee4")', "'forwards medium slide normal'");
-      shouldBe('webkitMarqueeValue("marquee5")', "'medium 10 slide normal'");
-      shouldBe('webkitMarqueeValue("marquee6")', "'forwards small normal'");
-      shouldBe('webkitMarqueeValue("marquee7")', "'forwards medium infinite'");
-      shouldBe('webkitMarqueeValue("marquee8")', "'slide fast'");
-      shouldBe('webkitMarqueeValue("marquee9")', "'forwards'");
-      shouldBe('webkitMarqueeValue("marquee10")', "'alternate'");
-      shouldBe('webkitMarqueeValue("marquee11")', "'infinite'");
-      debug("NOTE: 'foo' is an illegal CSS value for '-webkit-marquee'.");
-      shouldBeEqualToString('webkitMarqueeValue("marquee12")', "");
-      debug("NOTE: If only few longhand properties are specified, getPropertyValue for shorthand property returns null.")
-      shouldBeEqualToString('webkitMarqueeValue("marquee13")', "");
-      shouldBeEqualToString('webkitMarqueeValue("marquee14")', "");
-    </script>
-    <script src="../../resources/js-test-post.js"></script>
-  </body>
-</html>
index 5bcad8d..4188ef6 100644 (file)
@@ -27,7 +27,6 @@ PASS test('webkit-column-rule') is "inherit, initial"
 PASS test('flex-flow') is "inherit, initial"
 PASS test('flex') is "inherit, initial"
 PASS test('webkit-marginCollapse') is "inherit, initial"
-PASS test('webkit-marquee') is "inherit, initial"
 PASS test('webkit-mask') is "inherit, initial"
 PASS test('webkit-mask-position') is "inherit, initial"
 PASS test('webkit-mask-repeat') is "inherit, initial"
index b85fdb6..62d098f 100644 (file)
@@ -45,7 +45,6 @@ var shorthandNames = [
     "flex-flow",
     "flex",
     "webkit-marginCollapse",
-    "webkit-marquee",
     "webkit-mask",
     "webkit-mask-position",
     "webkit-mask-repeat",
index cabfdcf..e8ce786 100644 (file)
@@ -56,9 +56,6 @@ and adds "".
 Removing -webkit-margin-collapse
 removes "-webkit-margin-before-collapse, -webkit-margin-after-collapse"
 and adds "".
-Removing -webkit-marquee
-removes "-webkit-marquee-direction, -webkit-marquee-increment, -webkit-marquee-repetition, -webkit-marquee-style, -webkit-marquee-speed"
-and adds "".
 Removing -webkit-mask
 removes "-webkit-mask"
 and adds "".
index 38b6065..42d8efc 100644 (file)
@@ -17,7 +17,6 @@
                  list-style: square outside url(/marker.png); \
                  margin: 5%; \
                  -webkit-margin-collapse: collapse; \
-                 -webkit-marquee: ahead small infinite slide slow; \
                  padding: 2em; \
                  -webkit-text-stroke: orange 1pt; \
                  -webkit-columns: 100px 3; \
@@ -64,7 +63,6 @@
                 "list-style",
                 "margin",
                 "-webkit-margin-collapse",
-                "-webkit-marquee",
                 "-webkit-mask",
                 "-webkit-mask-position",
                 "overflow",
index 2b5070f..29ff284 100644 (file)
@@ -81,10 +81,6 @@ var propertiesToTest = {
     "-webkit-locale": true,
     "-webkit-margin-after-collapse": true,
     "-webkit-margin-before-collapse": true,
-    "-webkit-marquee-direction": true,
-    "-webkit-marquee-increment": true,
-    "-webkit-marquee-repetition": true,
-    "-webkit-marquee-style": true,
     "-webkit-mask-attachment": true,
     "-webkit-mask-box-image": true,
     "-webkit-mask-box-image-outset": true,
index 35c26a5..dc490a9 100644 (file)
@@ -234,10 +234,6 @@ zoom: 1;
 -webkit-locale: auto;
 -webkit-margin-after-collapse: collapse;
 -webkit-margin-before-collapse: collapse;
--webkit-marquee-direction: auto;
--webkit-marquee-increment: 6px;
--webkit-marquee-repetition: infinite;
--webkit-marquee-style: scroll;
 -webkit-mask-box-image: none;
 -webkit-mask-box-image-outset: 0px;
 -webkit-mask-box-image-repeat: stretch;
index 4ceb6f4..322fc5a 100644 (file)
@@ -233,10 +233,6 @@ zoom: 1
 -webkit-locale: auto
 -webkit-margin-after-collapse: collapse
 -webkit-margin-before-collapse: collapse
--webkit-marquee-direction: auto
--webkit-marquee-increment: 6px
--webkit-marquee-repetition: infinite
--webkit-marquee-style: scroll
 -webkit-mask-box-image: none
 -webkit-mask-box-image-outset: 0px
 -webkit-mask-box-image-repeat: stretch
index 2d98d3f..43c7191 100644 (file)
@@ -466,14 +466,6 @@ rect: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-direction) : auto
-rect: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-increment) : 6px
-rect: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-rect: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-style) : scroll
-rect: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image) : none
 rect: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
@@ -998,14 +990,6 @@ g: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-direction) : auto
-g: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-increment) : 6px
-g: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-g: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-style) : scroll
-g: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image) : none
 g: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
index 35c26a5..dc490a9 100644 (file)
@@ -234,10 +234,6 @@ zoom: 1;
 -webkit-locale: auto;
 -webkit-margin-after-collapse: collapse;
 -webkit-margin-before-collapse: collapse;
--webkit-marquee-direction: auto;
--webkit-marquee-increment: 6px;
--webkit-marquee-repetition: infinite;
--webkit-marquee-style: scroll;
 -webkit-mask-box-image: none;
 -webkit-mask-box-image-outset: 0px;
 -webkit-mask-box-image-repeat: stretch;
index 4ceb6f4..322fc5a 100644 (file)
@@ -233,10 +233,6 @@ zoom: 1
 -webkit-locale: auto
 -webkit-margin-after-collapse: collapse
 -webkit-margin-before-collapse: collapse
--webkit-marquee-direction: auto
--webkit-marquee-increment: 6px
--webkit-marquee-repetition: infinite
--webkit-marquee-style: scroll
 -webkit-mask-box-image: none
 -webkit-mask-box-image-outset: 0px
 -webkit-mask-box-image-repeat: stretch
index 2d98d3f..43c7191 100644 (file)
@@ -466,14 +466,6 @@ rect: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-direction) : auto
-rect: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-increment) : 6px
-rect: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-rect: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-style) : scroll
-rect: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image) : none
 rect: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
@@ -998,14 +990,6 @@ g: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-direction) : auto
-g: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-increment) : 6px
-g: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-g: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-style) : scroll
-g: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image) : none
 g: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
index 35c26a5..dc490a9 100644 (file)
@@ -234,10 +234,6 @@ zoom: 1;
 -webkit-locale: auto;
 -webkit-margin-after-collapse: collapse;
 -webkit-margin-before-collapse: collapse;
--webkit-marquee-direction: auto;
--webkit-marquee-increment: 6px;
--webkit-marquee-repetition: infinite;
--webkit-marquee-style: scroll;
 -webkit-mask-box-image: none;
 -webkit-mask-box-image-outset: 0px;
 -webkit-mask-box-image-repeat: stretch;
index 4ceb6f4..322fc5a 100644 (file)
@@ -233,10 +233,6 @@ zoom: 1
 -webkit-locale: auto
 -webkit-margin-after-collapse: collapse
 -webkit-margin-before-collapse: collapse
--webkit-marquee-direction: auto
--webkit-marquee-increment: 6px
--webkit-marquee-repetition: infinite
--webkit-marquee-style: scroll
 -webkit-mask-box-image: none
 -webkit-mask-box-image-outset: 0px
 -webkit-mask-box-image-repeat: stretch
index 2d98d3f..43c7191 100644 (file)
@@ -466,14 +466,6 @@ rect: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-direction) : auto
-rect: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-increment) : 6px
-rect: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-rect: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-style) : scroll
-rect: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image) : none
 rect: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
@@ -998,14 +990,6 @@ g: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-direction) : auto
-g: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-increment) : 6px
-g: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-g: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-style) : scroll
-g: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image) : none
 g: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
index 35c26a5..dc490a9 100644 (file)
@@ -234,10 +234,6 @@ zoom: 1;
 -webkit-locale: auto;
 -webkit-margin-after-collapse: collapse;
 -webkit-margin-before-collapse: collapse;
--webkit-marquee-direction: auto;
--webkit-marquee-increment: 6px;
--webkit-marquee-repetition: infinite;
--webkit-marquee-style: scroll;
 -webkit-mask-box-image: none;
 -webkit-mask-box-image-outset: 0px;
 -webkit-mask-box-image-repeat: stretch;
index 4ceb6f4..322fc5a 100644 (file)
@@ -233,10 +233,6 @@ zoom: 1
 -webkit-locale: auto
 -webkit-margin-after-collapse: collapse
 -webkit-margin-before-collapse: collapse
--webkit-marquee-direction: auto
--webkit-marquee-increment: 6px
--webkit-marquee-repetition: infinite
--webkit-marquee-style: scroll
 -webkit-mask-box-image: none
 -webkit-mask-box-image-outset: 0px
 -webkit-mask-box-image-repeat: stretch
index 2d98d3f..43c7191 100644 (file)
@@ -466,14 +466,6 @@ rect: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-direction) : auto
-rect: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-increment) : 6px
-rect: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-rect: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-style) : scroll
-rect: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image) : none
 rect: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
@@ -998,14 +990,6 @@ g: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-direction) : auto
-g: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-increment) : 6px
-g: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-g: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-style) : scroll
-g: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image) : none
 g: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
index b931abf..c4c71ba 100644 (file)
@@ -464,14 +464,6 @@ rect: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 rect: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-direction) : auto
-rect: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-increment) : 6px
-rect: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-rect: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-marquee-style) : scroll
-rect: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image) : none
 rect: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
@@ -994,14 +986,6 @@ g: style.getPropertyValue(-webkit-margin-after-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-after-collapse) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-margin-before-collapse) : collapse
 g: style.getPropertyCSSValue(-webkit-margin-before-collapse) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-direction) : auto
-g: style.getPropertyCSSValue(-webkit-marquee-direction) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-increment) : 6px
-g: style.getPropertyCSSValue(-webkit-marquee-increment) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-repetition) : infinite
-g: style.getPropertyCSSValue(-webkit-marquee-repetition) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-marquee-style) : scroll
-g: style.getPropertyCSSValue(-webkit-marquee-style) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image) : none
 g: style.getPropertyCSSValue(-webkit-mask-box-image) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-mask-box-image-outset) : 0px
index 898157b..0f4d5e6 100644 (file)
@@ -1,3 +1,31 @@
+2019-12-15  Emilio Cobos Álvarez  <emilio@crisal.io>
+
+        Remove -webkit-marquee.
+        https://bugs.webkit.org/show_bug.cgi?id=117769
+
+        Reviewed by Simon Fraser.
+
+        This doesn't simplify the code so much yet but makes the CSS properties not
+        accessible by web content, which means that how marquee is implemented is now an
+        implementation detail that WebKit can change.
+
+        Had to keep some of parsing code because addHTMLLengthToStyle uses the
+        CSS parser, which is a bit unfortunate. But we can avoid dealing with
+        identifiers so it can be simplified a bit, and similarly we can avoid
+        having a shorthand altogether.
+
+        Covered by existing tests that are being modified to reflect the
+        change.
+
+        * css/CSSComputedStyleDeclaration.cpp:
+        (WebCore::ComputedStyleExtractor::valueForPropertyInStyle):
+        * css/CSSProperties.json:
+        * css/StyleProperties.cpp:
+        (WebCore::StyleProperties::getPropertyValue const):
+        * css/parser/CSSPropertyParser.cpp:
+        (WebCore::CSSPropertyParser::parseSingleValue):
+        (WebCore::CSSPropertyParser::parseShorthand):
+
 2019-12-14  Adrian Perez de Castro  <aperez@igalia.com>
 
         [GTK][WPE] Fix various non-unified build issues introduced since r251698
index 1c4e36f..a757121 100644 (file)
@@ -2949,16 +2949,6 @@ RefPtr<CSSValue> ComputedStyleExtractor::valueForPropertyInStyle(const RenderSty
             return zoomAdjustedPaddingOrMarginPixelValue<&RenderStyle::marginBottom, &RenderBoxModelObject::marginBottom>(style, renderer);
         case CSSPropertyMarginLeft:
             return zoomAdjustedPaddingOrMarginPixelValue<&RenderStyle::marginLeft, &RenderBoxModelObject::marginLeft>(style, renderer);
-        case CSSPropertyWebkitMarqueeDirection:
-            return cssValuePool.createValue(style.marqueeDirection());
-        case CSSPropertyWebkitMarqueeIncrement:
-            return cssValuePool.createValue(style.marqueeIncrement());
-        case CSSPropertyWebkitMarqueeRepetition:
-            if (style.marqueeLoopCount() < 0)
-                return cssValuePool.createIdentifierValue(CSSValueInfinite);
-            return cssValuePool.createValue(style.marqueeLoopCount(), CSSUnitType::CSS_NUMBER);
-        case CSSPropertyWebkitMarqueeStyle:
-            return cssValuePool.createValue(style.marqueeBehavior());
         case CSSPropertyWebkitUserModify:
             return cssValuePool.createValue(style.userModify());
         case CSSPropertyMaxHeight: {
@@ -3812,7 +3802,10 @@ RefPtr<CSSValue> ComputedStyleExtractor::valueForPropertyInStyle(const RenderSty
         /* Unimplemented -webkit- properties */
         case CSSPropertyWebkitBorderRadius:
         case CSSPropertyWebkitMarginCollapse:
-        case CSSPropertyWebkitMarquee:
+        case CSSPropertyWebkitMarqueeDirection:
+        case CSSPropertyWebkitMarqueeIncrement:
+        case CSSPropertyWebkitMarqueeRepetition:
+        case CSSPropertyWebkitMarqueeStyle:
         case CSSPropertyWebkitMarqueeSpeed:
         case CSSPropertyWebkitMask:
         case CSSPropertyWebkitMaskRepeatX:
index 41578a9..b247245 100644 (file)
                 "status": "experimental"
             }
         },
-        "-webkit-marquee": {
+        "-webkit-marquee-direction": {
             "codegen-properties": {
-                "longhands": [
-                    "-webkit-marquee-direction",
-                    "-webkit-marquee-increment",
-                    "-webkit-marquee-repetition",
-                    "-webkit-marquee-style",
-                    "-webkit-marquee-speed"
-                ]
+                "internal-only": true
             },
-            "status": "non-standard"
-        },
-        "-webkit-marquee-direction": {
             "values": [
                 "forwards",
                 "backwards",
         },
         "-webkit-marquee-increment": {
             "codegen-properties": {
-                "conditional-converter": "MarqueeIncrement"
+                "conditional-converter": "MarqueeIncrement",
+                "internal-only": true
             },
             "status": "non-standard"
         },
         "-webkit-marquee-repetition": {
             "codegen-properties": {
                 "name-for-methods": "MarqueeLoopCount",
-                "converter": "MarqueeRepetition"
+                "converter": "MarqueeRepetition",
+                "internal-only": true
             },
             "status": "non-standard"
         },
         "-webkit-marquee-speed": {
             "codegen-properties": {
-                "converter": "MarqueeSpeed"
+                "converter": "MarqueeSpeed",
+                "internal-only": true
             },
             "status": "non-standard"
         },
                 "alternate"
             ],
             "codegen-properties": {
-                "name-for-methods": "MarqueeBehavior"
+                "name-for-methods": "MarqueeBehavior",
+                "internal-only": true
             },
             "status": "non-standard"
         },
index 8ac532f..5e9974f 100644 (file)
@@ -267,8 +267,6 @@ String StyleProperties::getPropertyValue(CSSPropertyID propertyID) const
         return getLayeredShorthandValue(transitionShorthand());
     case CSSPropertyListStyle:
         return getShorthandValue(listStyleShorthand());
-    case CSSPropertyWebkitMarquee:
-        return getShorthandValue(webkitMarqueeShorthand());
     case CSSPropertyWebkitMaskPosition:
         return getLayeredShorthandValue(webkitMaskPositionShorthand());
     case CSSPropertyWebkitMaskRepeat:
index ebd0eaf..5a7a178 100644 (file)
@@ -3657,22 +3657,16 @@ static RefPtr<CSSValue> consumeHangingPunctuation(CSSParserTokenRange& range)
 
 static RefPtr<CSSValue> consumeWebkitMarqueeIncrement(CSSParserTokenRange& range, CSSParserMode cssParserMode)
 {
-    if (range.peek().type() == IdentToken)
-        return consumeIdent<CSSValueSmall, CSSValueMedium, CSSValueLarge>(range);
     return consumeLengthOrPercent(range, cssParserMode, ValueRangeAll, UnitlessQuirk::Allow);
 }
 
 static RefPtr<CSSValue> consumeWebkitMarqueeRepetition(CSSParserTokenRange& range)
 {
-    if (range.peek().type() == IdentToken)
-        return consumeIdent<CSSValueInfinite>(range);
     return consumeNumber(range, ValueRangeNonNegative);
 }
 
 static RefPtr<CSSValue> consumeWebkitMarqueeSpeed(CSSParserTokenRange& range, CSSParserMode cssParserMode)
 {
-    if (range.peek().type() == IdentToken)
-        return consumeIdent<CSSValueSlow, CSSValueNormal, CSSValueFast>(range);
     return consumeTime(range, cssParserMode, ValueRangeNonNegative, UnitlessQuirk::Allow);
 }
 
@@ -5845,8 +5839,6 @@ bool CSSPropertyParser::parseShorthand(CSSPropertyID property, bool important)
         return consumePlaceItemsShorthand(important);
     case CSSPropertyPlaceSelf:
         return consumePlaceSelfShorthand(important);
-    case CSSPropertyWebkitMarquee:
-        return consumeShorthandGreedily(webkitMarqueeShorthand(), important);
     default:
         return false;
     }
index 7f3f0cb..97936ce 100644 (file)
@@ -654,30 +654,11 @@ inline int BuilderConverter::convertMarqueeRepetition(BuilderState&, const CSSVa
 inline int BuilderConverter::convertMarqueeSpeed(BuilderState&, const CSSValue& value)
 {
     auto& primitiveValue = downcast<CSSPrimitiveValue>(value);
-    int speed = 85;
-    if (CSSValueID ident = primitiveValue.valueID()) {
-        switch (ident) {
-        case CSSValueSlow:
-            speed = 500; // 500 msec.
-            break;
-        case CSSValueNormal:
-            speed = 85; // 85msec. The WinIE default.
-            break;
-        case CSSValueFast:
-            speed = 10; // 10msec. Super fast.
-            break;
-        default:
-            ASSERT_NOT_REACHED();
-            break;
-        }
-    } else if (primitiveValue.isTime())
-        speed = primitiveValue.computeTime<int, CSSPrimitiveValue::Milliseconds>();
-    else {
-        // For scrollamount support.
-        ASSERT(primitiveValue.isNumber());
-        speed = primitiveValue.intValue();
-    }
-    return speed;
+    if (primitiveValue.isTime())
+        return primitiveValue.computeTime<int, CSSPrimitiveValue::Milliseconds>();
+    // For scrollamount support.
+    ASSERT(primitiveValue.isNumber());
+    return primitiveValue.intValue();
 }
 
 inline Ref<QuotesData> BuilderConverter::convertQuotes(BuilderState&, const CSSValue& value)
@@ -1222,28 +1203,10 @@ inline Optional<float> BuilderConverter::convertPerspective(BuilderState& builde
 
 inline Optional<Length> BuilderConverter::convertMarqueeIncrement(BuilderState& builderState, const CSSValue& value)
 {
-    Optional<Length> marqueeLength;
-    auto& primitiveValue = downcast<CSSPrimitiveValue>(value);
-    switch (primitiveValue.valueID()) {
-    case CSSValueSmall:
-        marqueeLength = Length(1, Fixed); // 1px.
-        break;
-    case CSSValueNormal:
-        marqueeLength = Length(6, Fixed); // 6px. The WinIE default.
-        break;
-    case CSSValueLarge:
-        marqueeLength = Length(36, Fixed); // 36px.
-        break;
-    case CSSValueInvalid: {
-        Length length = primitiveValue.convertToLength<FixedIntegerConversion | PercentConversion | CalculatedConversion>(builderState.cssToLengthConversionData().copyWithAdjustedZoom(1.0f));
-        if (!length.isUndefined())
-            marqueeLength = length;
-        break;
-    }
-    default:
-        break;
-    }
-    return marqueeLength;
+    Length length = downcast<CSSPrimitiveValue>(value).convertToLength<FixedIntegerConversion | PercentConversion | CalculatedConversion>(builderState.cssToLengthConversionData().copyWithAdjustedZoom(1.0f));
+    if (length.isUndefined())
+        return WTF::nullopt;
+    return length;
 }
 
 inline Optional<FilterOperations> BuilderConverter::convertFilterOperations(BuilderState& builderState, const CSSValue& value)
index de4b676..60ee575 100644 (file)
@@ -1,3 +1,12 @@
+2019-12-15  Emilio Cobos Álvarez  <emilio@crisal.io>
+
+        Remove -webkit-marquee.
+        https://bugs.webkit.org/show_bug.cgi?id=117769
+
+        Reviewed by Simon Fraser.
+
+        * UserInterface/Models/CSSKeywordCompletions.js:
+
 2019-12-12  Yury Semikhatsky  <yurys@chromium.org>
 
         Web Inspector: Error: Can't make a ContentView for an unknown representedObject of type: CallFrame
index 5eb614c..3900f16 100644 (file)
@@ -612,15 +612,6 @@ WI.CSSKeywordCompletions._propertyKeywordMap = {
     "perspective-origin": [
         "none", "left", "right", "bottom", "top", "center"
     ],
-    "-webkit-marquee-increment": [
-        "small", "large", "medium"
-    ],
-    "-webkit-marquee-repetition": [
-        "infinite"
-    ],
-    "-webkit-marquee-speed": [
-        "normal", "slow", "fast"
-    ],
     "margin-right": [
         "auto"
     ],
@@ -953,12 +944,6 @@ WI.CSSKeywordCompletions._propertyKeywordMap = {
     "-webkit-margin-top-collapse": [
         "collapse", "separate", "discard",
     ],
-    "-webkit-marquee-direction": [
-        "left", "right", "auto", "reverse", "forwards", "backwards", "ahead", "up", "down",
-    ],
-    "-webkit-marquee-style": [
-        "none", "scroll", "slide", "alternate",
-    ],
     "-webkit-max-logical-height": [
         "auto", "intrinsic", "min-intrinsic", "min-content", "-webkit-min-content", "max-content", "-webkit-max-content", "-webkit-fill-available", "fit-content", "-webkit-fit-content", "none", "calc()"
     ],