Convert CSSProperties.in to JSON5 format

third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_converter.py

Index: third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_converter.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_converter.py b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_converter.py
index cdf326caa7094d8c9bdb4c890c91522e93be6b62..dfedfd4a608b0fb16899ac6a38f4436f7ff4170c 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_converter.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/test_converter.py
@@ -87,8 +87,9 @@ class _W3CTestConverter(HTMLParser):
         self.resources_relpath = resources_relpath
 
         # These settings might vary between WebKit and Blink.
-        self._css_property_file = self.path_from_webkit_root('Source', 'core', 'css', 'CSSProperties.in')
-        self.prefixed_properties = self.read_webkit_prefixed_css_property_list()
+        # Only -webkit-text-emphasis is currently needed. See:
+        # https://bugs.chromium.org/p/chromium/issues/detail?id=614955#c1
+        self.prefixed_properties = ['-webkit-text-emphasis']

[tkent, 2017/01/17 03:14:53]

I'm sorry, but we need more property names:
 -webkit-text-emphasis
 -webkit-text-emphasis-style
 -webkit-text-emphasis-position

https://cs.chromium.org/search/?q=webkit-+file:imported/csswg-test&sq=package...

[tkent, 2017/01/17 03:16:50]

-webkit-text-emphasis-color too :)

[ktyliu, 2017/01/17 03:50:25]

Done :)

         prop_regex = r'([\s{]|^)(' + '|'.join(
             prop.replace('-webkit-', '') for prop in self.prefixed_properties) + r')(\s+:|:)'
         self.prop_re = re.compile(prop_regex)
@@ -99,26 +100,6 @@ class _W3CTestConverter(HTMLParser):
     def path_from_webkit_root(self, *comps):
         return self._filesystem.abspath(self._filesystem.join(self._webkit_root, *comps))
 
-    def read_webkit_prefixed_css_property_list(self):
-        prefixed_properties = []
-        unprefixed_properties = set()
-
-        contents = self._filesystem.read_text_file(self._css_property_file)
-        for line in contents.splitlines():
-            if re.match(r'^(#|//|$)', line):
-                # skip comments and preprocessor directives.
-                continue
-            prop = line.split()[0]
-            # Find properties starting with the -webkit- prefix.
-            match = re.match(r'-webkit-([\w|-]*)', prop)
-            if match:
-                prefixed_properties.append(match.group(1))
-            else:
-                unprefixed_properties.add(prop.strip())
-
-        # Ignore any prefixed properties for which an unprefixed version is supported.
-        return [prop for prop in prefixed_properties if prop not in unprefixed_properties]
-
     def add_webkit_prefix_to_unprefixed_properties(self, text):
         """Searches |text| for instances of properties requiring the -webkit- prefix and adds the prefix to them.