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

Unified Diff: chrome/common/extensions/features/chrome_channel_feature_filter_unittest.cc

Issue 2030013003: Remove ListValue::Append(new {Fundamental,String}Value(...)) pattern in //chrome (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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: chrome/common/extensions/features/chrome_channel_feature_filter_unittest.cc
diff --git a/chrome/common/extensions/features/chrome_channel_feature_filter_unittest.cc b/chrome/common/extensions/features/chrome_channel_feature_filter_unittest.cc
index 5367ab397d4160f6213e901237dd59397d09ba36..806042b5d31f260aef5f21a07cf6713eabfbcf46 100644
--- a/chrome/common/extensions/features/chrome_channel_feature_filter_unittest.cc
+++ b/chrome/common/extensions/features/chrome_channel_feature_filter_unittest.cc
@@ -139,10 +139,10 @@ TEST_F(ChromeChannelFeatureFilterTest, FeatureValidation) {
base::DictionaryValue* feature2 = new base::DictionaryValue();
feature2->SetString("channel", "trunk");
base::ListValue* extension_types = new base::ListValue();
- extension_types->Append(new base::StringValue("extension"));
+ extension_types->AppendString("extension");
feature2->Set("extension_types", extension_types);
base::ListValue* contexts = new base::ListValue();
- contexts->Append(new base::StringValue("blessed_extension"));
+ contexts->AppendString("blessed_extension");
feature2->Set("contexts", contexts);
value->Set("feature2", feature2);

Powered by Google App Engine
This is Rietveld 408576698