Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1016)

Unified Diff: third_party/WebKit/Source/bindings/scripts/v8_interface.py

Issue 2116503004: Make Foreign Fetch an origin trial. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/bindings/scripts/v8_interface.py
diff --git a/third_party/WebKit/Source/bindings/scripts/v8_interface.py b/third_party/WebKit/Source/bindings/scripts/v8_interface.py
index f2b579de884c0231840db9f0a5db30e29fe19174..fc3e2c467b93426f6de6d1e642bc3e8e9cca4610 100644
--- a/third_party/WebKit/Source/bindings/scripts/v8_interface.py
+++ b/third_party/WebKit/Source/bindings/scripts/v8_interface.py
@@ -116,7 +116,8 @@ def origin_trial_features(interface, constants, attributes, methods):
method['origin_trial_feature_name'])]
)
# Group members by origin_trial_feature_name
- members_by_name = itertools.groupby(origin_trial_members, itemgetter('origin_trial_feature_name'))
+ members_by_name = itertools.groupby(sorted(origin_trial_members, key=itemgetter('origin_trial_feature_name')), itemgetter('origin_trial_feature_name'))
+
# 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,

Powered by Google App Engine
This is Rietveld 408576698