Remove ListValue::Append(new {Fundamental,String}Value(...)) pattern in //extensions

extensions/browser/api/storage/settings_test_util.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "extensions/browser/api/storage/settings_test_util.h"
 
 #include "base/files/file_path.h"
 #include "extensions/browser/api/storage/storage_frontend.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/extension_system_provider.h"
@@ skipping 56 matching lines @@
     const std::string& id,
     Manifest::Type type,
     const std::set<std::string>& permissions_set) {
   base::DictionaryValue manifest;
   manifest.SetString("name", std::string("Test extension ") + id);
   manifest.SetString("version", "1.0");
 
   std::unique_ptr<base::ListValue> permissions(new base::ListValue());
   for (std::set<std::string>::const_iterator it = permissions_set.begin();
       it != permissions_set.end(); ++it) {
-    permissions->Append(new base::StringValue(*it));
+    permissions->AppendString(*it);
   }
   manifest.Set("permissions", permissions.release());
 
   switch (type) {
     case Manifest::TYPE_EXTENSION:
       break;
 
     case Manifest::TYPE_LEGACY_PACKAGED_APP: {
       base::DictionaryValue* app = new base::DictionaryValue();
       base::DictionaryValue* app_launch = new base::DictionaryValue();
@@ skipping 26 matching lines @@
       it != permissions_set.end(); ++it) {
     DCHECK(extension->permissions_data()->HasAPIPermission(*it));
   }
 
   return extension;
 }
 
 }  // namespace settings_test_util
 
 }  // namespace extensions