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

Side by Side Diff: base/feature_list.cc

Issue 1306653004: Expand FeatureList to support FieldTrial association. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: overriden -> overridden; at() -> find() Created 5 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/feature_list.h" 5 #include "base/feature_list.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/metrics/field_trial.h"
10 #include "base/strings/string_split.h" 11 #include "base/strings/string_split.h"
11 12
12 namespace base { 13 namespace base {
13 14
14 namespace { 15 namespace {
15 16
16 // Pointer to the FeatureList instance singleton that was set via 17 // Pointer to the FeatureList instance singleton that was set via
17 // FeatureList::SetInstance(). Does not use base/memory/singleton.h in order to 18 // FeatureList::SetInstance(). Does not use base/memory/singleton.h in order to
18 // have more control over initialization timing. Leaky. 19 // have more control over initialization timing. Leaky.
19 FeatureList* g_instance = nullptr; 20 FeatureList* g_instance = nullptr;
(...skipping 10 matching lines...) Expand all
30 FeatureList::~FeatureList() {} 31 FeatureList::~FeatureList() {}
31 32
32 void FeatureList::InitializeFromCommandLine( 33 void FeatureList::InitializeFromCommandLine(
33 const std::string& enable_features, 34 const std::string& enable_features,
34 const std::string& disable_features) { 35 const std::string& disable_features) {
35 DCHECK(!initialized_); 36 DCHECK(!initialized_);
36 37
37 // Process disabled features first, so that disabled ones take precedence over 38 // Process disabled features first, so that disabled ones take precedence over
38 // enabled ones (since RegisterOverride() uses insert()). 39 // enabled ones (since RegisterOverride() uses insert()).
39 for (const auto& feature_name : SplitFeatureListString(disable_features)) { 40 for (const auto& feature_name : SplitFeatureListString(disable_features)) {
40 RegisterOverride(feature_name, OVERRIDE_DISABLE_FEATURE); 41 RegisterOverride(feature_name, OVERRIDE_DISABLE_FEATURE, nullptr);
41 } 42 }
42 for (const auto& feature_name : SplitFeatureListString(enable_features)) { 43 for (const auto& feature_name : SplitFeatureListString(enable_features)) {
43 RegisterOverride(feature_name, OVERRIDE_ENABLE_FEATURE); 44 RegisterOverride(feature_name, OVERRIDE_ENABLE_FEATURE, nullptr);
44 } 45 }
45 } 46 }
46 47
48 bool FeatureList::IsFeatureOverriddenFromCommandLine(
49 const std::string& feature_name,
50 OverrideState state) const {
51 auto it = overrides_.find(feature_name);
52 return it != overrides_.end() && it->second.overridden_state == state &&
53 !it->second.overridden_by_field_trial;
54 }
55
56 void FeatureList::RegisterFieldTrialOverride(const std::string& feature_name,
57 OverrideState override_state,
58 FieldTrial* field_trial) {
59 DCHECK(field_trial);
60 DCHECK(!ContainsKey(overrides_, feature_name) ||
61 !overrides_.find(feature_name)->second.field_trial)
62 << "Feature " << feature_name
63 << " has conflicting field trial overrides: "
64 << overrides_.find(feature_name)->second.field_trial->trial_name()
65 << " / " << field_trial->trial_name();
66
67 RegisterOverride(feature_name, override_state, field_trial);
68 }
69
70 void FeatureList::AssociateReportingFieldTrial(
71 const std::string& feature_name,
72 OverrideState for_overridden_state,
73 FieldTrial* field_trial) {
74 DCHECK(
75 IsFeatureOverriddenFromCommandLine(feature_name, for_overridden_state));
76
77 // Only one associated field trial is supported per feature. This is generally
78 // enforced server-side.
79 OverrideEntry* entry = &overrides_.find(feature_name)->second;
80 if (entry->field_trial) {
81 NOTREACHED() << "Feature " << feature_name
82 << " already has trial: " << entry->field_trial->trial_name()
83 << ", associating trial: " << field_trial->trial_name();
84 return;
85 }
86
87 entry->field_trial = field_trial;
88 }
89
47 // static 90 // static
48 bool FeatureList::IsEnabled(const Feature& feature) { 91 bool FeatureList::IsEnabled(const Feature& feature) {
49 return GetInstance()->IsFeatureEnabled(feature); 92 return GetInstance()->IsFeatureEnabled(feature);
50 } 93 }
51 94
52 // static 95 // static
53 FeatureList* FeatureList::GetInstance() { 96 FeatureList* FeatureList::GetInstance() {
54 return g_instance; 97 return g_instance;
55 } 98 }
56 99
(...skipping 17 matching lines...) Expand all
74 initialized_ = true; 117 initialized_ = true;
75 } 118 }
76 119
77 bool FeatureList::IsFeatureEnabled(const Feature& feature) { 120 bool FeatureList::IsFeatureEnabled(const Feature& feature) {
78 DCHECK(initialized_); 121 DCHECK(initialized_);
79 DCHECK(CheckFeatureIdentity(feature)) << feature.name; 122 DCHECK(CheckFeatureIdentity(feature)) << feature.name;
80 123
81 auto it = overrides_.find(feature.name); 124 auto it = overrides_.find(feature.name);
82 if (it != overrides_.end()) { 125 if (it != overrides_.end()) {
83 const OverrideEntry& entry = it->second; 126 const OverrideEntry& entry = it->second;
127
128 // Activate the corresponding field trial, if necessary.
129 if (entry.field_trial)
130 entry.field_trial->group();
131
84 // TODO(asvitkine) Expand this section as more support is added. 132 // TODO(asvitkine) Expand this section as more support is added.
85 return entry.overridden_state == OVERRIDE_ENABLE_FEATURE; 133 return entry.overridden_state == OVERRIDE_ENABLE_FEATURE;
86 } 134 }
87 // Otherwise, return the default state. 135 // Otherwise, return the default state.
88 return feature.default_state == FEATURE_ENABLED_BY_DEFAULT; 136 return feature.default_state == FEATURE_ENABLED_BY_DEFAULT;
89 } 137 }
90 138
91 void FeatureList::RegisterOverride(const std::string& feature_name, 139 void FeatureList::RegisterOverride(const std::string& feature_name,
92 OverrideState overridden_state) { 140 OverrideState overridden_state,
141 FieldTrial* field_trial) {
93 DCHECK(!initialized_); 142 DCHECK(!initialized_);
94 overrides_.insert(make_pair(feature_name, OverrideEntry(overridden_state))); 143 // Note: The semantics of insert() is that it does not overwrite the entry if
144 // one already exists for the key. Thus, only the first override for a given
145 // feature name takes effect.
146 overrides_.insert(std::make_pair(
147 feature_name, OverrideEntry(overridden_state, field_trial)));
95 } 148 }
96 149
97 bool FeatureList::CheckFeatureIdentity(const Feature& feature) { 150 bool FeatureList::CheckFeatureIdentity(const Feature& feature) {
98 AutoLock auto_lock(feature_identity_tracker_lock_); 151 AutoLock auto_lock(feature_identity_tracker_lock_);
99 152
100 auto it = feature_identity_tracker_.find(feature.name); 153 auto it = feature_identity_tracker_.find(feature.name);
101 if (it == feature_identity_tracker_.end()) { 154 if (it == feature_identity_tracker_.end()) {
102 // If it's not tracked yet, register it. 155 // If it's not tracked yet, register it.
103 feature_identity_tracker_[feature.name] = &feature; 156 feature_identity_tracker_[feature.name] = &feature;
104 return true; 157 return true;
105 } 158 }
106 // Compare address of |feature| to the existing tracked entry. 159 // Compare address of |feature| to the existing tracked entry.
107 return it->second == &feature; 160 return it->second == &feature;
108 } 161 }
109 162
110 FeatureList::OverrideEntry::OverrideEntry(OverrideState overridden_state) 163 FeatureList::OverrideEntry::OverrideEntry(OverrideState overridden_state,
111 : overridden_state(overridden_state) {} 164 FieldTrial* field_trial)
165 : overridden_state(overridden_state),
166 field_trial(field_trial),
167 overridden_by_field_trial(field_trial != nullptr) {}
112 168
113 } // namespace base 169 } // namespace base
OLDNEW
« base/feature_list.h ('K') | « base/feature_list.h ('k') | base/feature_list_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698