Add 'priority' key to CSSProperties.in

third_party/WebKit/Source/build/scripts/css_properties.py

 #!/usr/bin/env python
 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import in_generator
 import name_utilities
 
 
 class CSSProperties(in_generator.Writer):
@@ skipping 13 matching lines @@
         'setter': None,
         'initial': None,
         'type_name': None,
         'converter': None,
         'custom_all': False,
         'custom_initial': False,
         'custom_inherit': False,
         'custom_value': False,
         'builder_skip': False,
         'direction_aware': False,
+        'priority': 2,
         'api_class': None,
         # Generated ComputedStyle annotations.
         'field_storage_type': None,
         # Typed OM annotations.
         'typedom_types': [],
         'keywords': [],
         'initial_keyword': None,
         'keyword_only': False,
         'supports_percentage': False,
         'repeated': False,
@@ skipping 11 matching lines @@
         'custom_value': (True, False),
         'builder_skip': (True, False),
         'direction_aware': (True, False),
         'keyword_only': (True, False),
     }
 
     def __init__(self, file_paths):
         in_generator.Writer.__init__(self, file_paths)
 
         properties = self.in_file.name_dictionaries
+
+        # Sort properties by priority, then alphabetically.
+        properties.sort(key=lambda p: (int(p['priority']), name_utilities.strip_webkit_prefix(p['name'])))

[alancutter (OOO until 2018), 2017/01/11 03:24:20]

This sorting key has collisions between different items resulting in potentially
non-deterministic behaviour.
I wouldn't bother stripping "-webkit-", alternatively just strip the "-" and
assert that no sorting key collisions occur.

[sashab, 2017/01/11 03:57:44]

Done. Added check for sorting key collisions.

+
         self._aliases = [property for property in properties if property['alias_for']]
         properties = [property for property in properties if not property['alias_for']]
 
         # 0: CSSPropertyInvalid
         # 1: CSSPropertyApplyAtRule
         # 2: CSSPropertyVariable
         self._first_enum_value = 3
 
         # StylePropertyMetadata additionally assumes there are under 1024 properties.
         assert self._first_enum_value + len(properties) < 512, 'Property aliasing expects there are under 512 properties.'
 
         for offset, property in enumerate(properties):
             property['property_id'] = name_utilities.enum_for_css_property(property['name'])
             property['upper_camel_name'] = name_utilities.camel_case(property['name'])
             property['lower_camel_name'] = name_utilities.lower_first(property['upper_camel_name'])
             property['enum_value'] = self._first_enum_value + offset
             property['is_internal'] = property['name'].startswith('-internal-')
 
         self._properties_including_aliases = properties
         self._properties = {property['property_id']: property for property in properties}
 
         # The generated code will only work with at most one alias per property
         assert len({property['alias_for'] for property in self._aliases}) == len(self._aliases)
 
         for property in self._aliases:
             property['property_id'] = name_utilities.enum_for_css_property_alias(property['name'])
             aliased_property = self._properties[name_utilities.enum_for_css_property(property['alias_for'])]
             property['enum_value'] = aliased_property['enum_value'] + 512
         self._properties_including_aliases += self._aliases