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
+
+        # Group properties by priority.
+        prioritised_properties = {0: [], 1: [], 2: []}

[ktyliu, 2017/01/11 02:45:43]

Note you can do this all in one statement by using sorting based on tuple:

# Sort by priority and then by stripped name.
propertie.sort(key = lambda p: (p.priority,
name_utilities.strip_webkit_prefix(p['name']))

[sashab, 2017/01/11 02:50:46]

"Let me do your entire patch in one line kthxbai"

;) Nice one!!!!

+        for property in properties:
+            priority = int(property['priority'])
+            assert priority in [0, 1, 2], 'Priority must be a number between 0 and 2'
+            prioritised_properties[priority].append(property)
+
+        # Sort each priority alphabetically and concatenate them.
+        property_name_key = lambda p: name_utilities.strip_webkit_prefix(p['name'])
+        properties = (sorted(prioritised_properties[0], key=property_name_key) +
+                      sorted(prioritised_properties[1], key=property_name_key) +
+                      sorted(prioritised_properties[2], key=property_name_key))
+
         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