OLD | NEW |
---|---|
1 // Copyright 2016 The Chromium Authors. All rights reserved. | 1 // Copyright 2016 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/test/scoped_feature_list.h" | 5 #include "base/test/scoped_feature_list.h" |
6 | 6 |
7 #include <algorithm> | |
7 #include <string> | 8 #include <string> |
9 #include <vector> | |
10 | |
11 #include "base/strings/string_split.h" | |
12 #include "base/strings/string_util.h" | |
8 | 13 |
9 namespace base { | 14 namespace base { |
10 namespace test { | 15 namespace test { |
11 | 16 |
12 namespace { | 17 namespace { |
13 | 18 |
14 static std::string GetFeatureString( | 19 std::vector<std::string> GetFeatureVector( |
15 const std::initializer_list<base::Feature>& features) { | 20 const std::initializer_list<base::Feature>& features) { |
16 std::string output; | 21 std::vector<std::string> output; |
Alexei Svitkine (slow)
2017/04/24 15:55:11
Can this be a StringPiece vector?
| |
17 for (const base::Feature& feature : features) { | 22 for (const base::Feature& feature : features) { |
18 if (!output.empty()) | 23 output.push_back(feature.name); |
19 output += ","; | |
20 output += feature.name; | |
21 } | 24 } |
25 | |
22 return output; | 26 return output; |
23 } | 27 } |
24 | 28 |
29 // Get feature name for feature string from FeatureList::GetFeatureOverrides. | |
Ilya Sherman
2017/04/24 19:49:51
nit: I'd rephrase this to something like
// Ext
| |
30 StringPiece GetFeatureName(StringPiece feature) { | |
31 StringPiece feature_name = feature; | |
32 | |
33 // Remove default info. | |
34 if (feature_name.starts_with("*")) | |
35 feature_name = feature_name.substr(1); | |
36 | |
37 // Remove field_trial info. | |
38 std::size_t index = feature_name.find("<"); | |
39 if (index != std::string::npos) | |
40 feature_name = feature_name.substr(0, index); | |
41 | |
42 return feature_name; | |
43 } | |
44 | |
45 struct Features { | |
46 std::vector<std::string> enabled_feature_list; | |
47 std::vector<std::string> disabled_feature_list; | |
48 }; | |
49 | |
50 // Given features from FeatureList::GetFeatureOverrides, add them to | |
51 // merged_features if it is not existing in merged_features. | |
Ilya Sherman
2017/04/24 19:49:51
The semantics of the params are not entirely obvio
| |
52 void OverrideFeatures(const std::string& features, | |
53 base::FeatureList::OverrideState override_state, | |
54 Features* merged_features) { | |
chaopeng
2017/04/22 03:18:15
Is this same as your comment?
Ilya Sherman
2017/04/24 19:49:51
It's not really what I had in mind, but I guess th
| |
55 std::vector<StringPiece> features_list = | |
56 SplitStringPiece(features, ",", TRIM_WHITESPACE, SPLIT_WANT_NONEMPTY); | |
57 | |
58 for (StringPiece feature : features_list) { | |
59 StringPiece feature_name = GetFeatureName(feature); | |
60 | |
61 if (std::count(merged_features->enabled_feature_list.begin(), | |
Alexei Svitkine (slow)
2017/04/24 15:55:11
Nit: Use ContainsValue() from base/stl_util.h
| |
62 merged_features->enabled_feature_list.end(), | |
63 feature_name) > 0) | |
64 continue; | |
65 | |
66 if (std::count(merged_features->disabled_feature_list.begin(), | |
67 merged_features->disabled_feature_list.end(), | |
68 feature_name) > 0) | |
69 continue; | |
70 | |
71 if (override_state == base::FeatureList::OVERRIDE_ENABLE_FEATURE) { | |
72 merged_features->enabled_feature_list.push_back(feature.as_string()); | |
73 } else { | |
74 merged_features->disabled_feature_list.push_back(feature.as_string()); | |
Alexei Svitkine (slow)
2017/04/24 15:55:11
DCHECK that it's OVERRIDE_DISABLE_FEATURE?
| |
75 } | |
76 } | |
77 } | |
78 | |
25 } // namespace | 79 } // namespace |
26 | 80 |
27 ScopedFeatureList::ScopedFeatureList() {} | 81 ScopedFeatureList::ScopedFeatureList() {} |
28 | 82 |
29 ScopedFeatureList::~ScopedFeatureList() { | 83 ScopedFeatureList::~ScopedFeatureList() { |
30 if (original_feature_list_) { | 84 if (original_feature_list_) { |
31 base::FeatureList::ClearInstanceForTesting(); | 85 base::FeatureList::ClearInstanceForTesting(); |
32 base::FeatureList::RestoreInstanceForTesting( | 86 base::FeatureList::RestoreInstanceForTesting( |
33 std::move(original_feature_list_)); | 87 std::move(original_feature_list_)); |
34 } | 88 } |
35 } | 89 } |
36 | 90 |
37 void ScopedFeatureList::Init() { | 91 void ScopedFeatureList::Init() { |
38 std::unique_ptr<base::FeatureList> feature_list(new base::FeatureList); | 92 std::unique_ptr<base::FeatureList> feature_list(new base::FeatureList); |
39 feature_list->InitializeFromCommandLine(std::string(), std::string()); | 93 feature_list->InitializeFromCommandLine(std::string(), std::string()); |
40 InitWithFeatureList(std::move(feature_list)); | 94 InitWithFeatureList(std::move(feature_list)); |
41 } | 95 } |
42 | 96 |
43 void ScopedFeatureList::InitWithFeatures( | |
44 const std::initializer_list<base::Feature>& enabled_features, | |
45 const std::initializer_list<base::Feature>& disabled_features) { | |
46 InitFromCommandLine(GetFeatureString(enabled_features), | |
47 GetFeatureString(disabled_features)); | |
48 } | |
49 | |
50 void ScopedFeatureList::InitWithFeatureList( | 97 void ScopedFeatureList::InitWithFeatureList( |
51 std::unique_ptr<FeatureList> feature_list) { | 98 std::unique_ptr<FeatureList> feature_list) { |
52 DCHECK(!original_feature_list_); | 99 DCHECK(!original_feature_list_); |
53 original_feature_list_ = base::FeatureList::ClearInstanceForTesting(); | 100 original_feature_list_ = base::FeatureList::ClearInstanceForTesting(); |
54 base::FeatureList::SetInstance(std::move(feature_list)); | 101 base::FeatureList::SetInstance(std::move(feature_list)); |
55 } | 102 } |
56 | 103 |
57 void ScopedFeatureList::InitFromCommandLine( | 104 void ScopedFeatureList::InitFromCommandLine( |
58 const std::string& enable_features, | 105 const std::string& enable_features, |
59 const std::string& disable_features) { | 106 const std::string& disable_features) { |
60 std::unique_ptr<base::FeatureList> feature_list(new base::FeatureList); | 107 std::unique_ptr<base::FeatureList> feature_list(new base::FeatureList); |
61 feature_list->InitializeFromCommandLine(enable_features, disable_features); | 108 feature_list->InitializeFromCommandLine(enable_features, disable_features); |
62 InitWithFeatureList(std::move(feature_list)); | 109 InitWithFeatureList(std::move(feature_list)); |
63 } | 110 } |
64 | 111 |
112 void ScopedFeatureList::InitWithFeatures( | |
113 const std::initializer_list<base::Feature>& enabled_features, | |
114 const std::initializer_list<base::Feature>& disabled_features) { | |
115 Features merged_features; | |
116 merged_features.enabled_feature_list = GetFeatureVector(enabled_features); | |
117 merged_features.disabled_feature_list = GetFeatureVector(disabled_features); | |
118 | |
119 base::FeatureList* feature_list = base::FeatureList::GetInstance(); | |
120 if (feature_list) { | |
121 std::string enabled_features; | |
122 std::string disabled_features; | |
123 base::FeatureList::GetInstance()->GetFeatureOverrides(&enabled_features, | |
124 &disabled_features); | |
125 OverrideFeatures(enabled_features, | |
126 FeatureList::OverrideState::OVERRIDE_ENABLE_FEATURE, | |
127 &merged_features); | |
128 OverrideFeatures(disabled_features, | |
129 FeatureList::OverrideState::OVERRIDE_DISABLE_FEATURE, | |
130 &merged_features); | |
131 } | |
132 | |
133 std::string enabled = JoinString(merged_features.enabled_feature_list, ","); | |
134 std::string disabled = JoinString(merged_features.disabled_feature_list, ","); | |
135 InitFromCommandLine(enabled, disabled); | |
136 } | |
137 | |
65 void ScopedFeatureList::InitAndEnableFeature(const base::Feature& feature) { | 138 void ScopedFeatureList::InitAndEnableFeature(const base::Feature& feature) { |
66 InitFromCommandLine(feature.name, std::string()); | 139 InitWithFeatures({feature}, {}); |
67 } | 140 } |
68 | 141 |
69 void ScopedFeatureList::InitAndDisableFeature(const base::Feature& feature) { | 142 void ScopedFeatureList::InitAndDisableFeature(const base::Feature& feature) { |
70 InitFromCommandLine(std::string(), feature.name); | 143 InitWithFeatures({}, {feature}); |
71 } | 144 } |
72 | 145 |
73 } // namespace test | 146 } // namespace test |
74 } // namespace base | 147 } // namespace base |
OLD | NEW |