[Bindings][Refactoring] Remove filters for origin trial features

third_party/WebKit/Source/bindings/scripts/v8_interface.py

 # Copyright (C) 2013 Google Inc. All rights reserved.
 # coding=utf-8
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
@@ skipping 89 matching lines @@
 
 
 def origin_trial_features(interface, constants, attributes, methods):
     """ Returns a list of the origin trial features used in this interface.
 
     Each element is a dictionary with keys 'name' and 'needs_instance'.
     'needs_instance' is true if any member associated with the interface needs
     to be installed on every instance of the interface. This list is the union
     of the sets of features used for constants, attributes and methods.
     """
+    key = 'origin_trial_feature_name'
 
     # Collect all members visible on this interface with a defined origin trial
-    origin_trial_members = (
-        [constant for constant in constants if constant['origin_trial_feature_name']] +
-        [attribute for attribute in attributes if attribute['origin_trial_feature_name']] +
-        [method for method in methods if (
-            v8_methods.method_is_visible(method, interface.is_partial) and
-            method['origin_trial_feature_name'])]
-    )
-    # Group members by origin_trial_feature_name
-    members_by_name = itertools.groupby(sorted(origin_trial_members,
-                                               key=itemgetter('origin_trial_feature_name')),
-                                        itemgetter('origin_trial_feature_name'))
+    origin_trial_constants = [constant for constant in constants if constant[key]]
+    origin_trial_attributes = [attribute for attribute in attributes if attribute[key]]
+    origin_trial_methods = [method for method in methods if (
+        v8_methods.method_is_visible(method, interface.is_partial) and method[key])]
+
+    feature_names = set([constant[key] for constant in origin_trial_constants] +
+                        [attribute[key] for attribute in origin_trial_attributes] +
+                        [method[key] for method in origin_trial_methods])
+
     # Construct the list of dictionaries. 'needs_instance' will be true if any
     # member for the feature has 'on_instance' defined as true.
     features = [{'name': name,
-                 'needs_instance': reduce(or_, (member.get('on_instance', False)
-                                                for member in members))}
-                for name, members in members_by_name]
+                 'constants': [constant for constant in origin_trial_constants if constant[key] == name],
+                 'attributes': [attribute for attribute in origin_trial_attributes if attribute[key] == name],
+                 'methods': [method for method in origin_trial_methods if method[key] == name]}
+                for name in feature_names]
+    for feature in features:
+        members = feature['constants'] + feature['attributes'] + feature['methods']
+        feature['needs_instance'] = reduce(or_, (member.get('on_instance', False) for member in members))
+
     if features:
         includes.add('bindings/core/v8/ScriptState.h')
         includes.add('core/origin_trials/OriginTrials.h')
     return sorted(features)
 
 
 def interface_context(interface, interfaces):
     """Creates a Jinja template context for an interface.
 
     Args:
@@ skipping 1350 matching lines @@
     extended_attributes = deleter.extended_attributes
     is_call_with_script_state = v8_utilities.has_extended_attribute_value(deleter, 'CallWith', 'ScriptState')
     is_ce_reactions = 'CEReactions' in extended_attributes
     return {
         'is_call_with_script_state': is_call_with_script_state,
         'is_ce_reactions': is_ce_reactions,
         'is_custom': 'Custom' in extended_attributes,
         'is_raises_exception': 'RaisesException' in extended_attributes,
         'name': cpp_name(deleter),
     }