Index: extensions/browser/api/storage/settings_test_util.cc |
diff --git a/extensions/browser/api/storage/settings_test_util.cc b/extensions/browser/api/storage/settings_test_util.cc |
index 1b619ef6f1b15c533395ff2586bec191bb2bb6da..de30f6a09d343393b0ea4674194b3b607e088926 100644 |
--- a/extensions/browser/api/storage/settings_test_util.cc |
+++ b/extensions/browser/api/storage/settings_test_util.cc |
@@ -4,8 +4,12 @@ |
#include "extensions/browser/api/storage/settings_test_util.h" |
+#include <utility> |
+ |
#include "base/files/file_path.h" |
+#include "base/memory/ptr_util.h" |
#include "base/run_loop.h" |
+#include "base/values.h" |
#include "extensions/browser/api/storage/storage_frontend.h" |
#include "extensions/browser/extension_registry.h" |
#include "extensions/browser/extension_system_provider.h" |
@@ -77,18 +81,18 @@ scoped_refptr<const Extension> AddExtensionWithIdAndPermissions( |
it != permissions_set.end(); ++it) { |
permissions->AppendString(*it); |
} |
- manifest.Set("permissions", permissions.release()); |
+ manifest.Set("permissions", std::move(permissions)); |
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(); |
+ auto app = base::MakeUnique<base::DictionaryValue>(); |
+ auto app_launch = base::MakeUnique<base::DictionaryValue>(); |
app_launch->SetString("local_path", "fake.html"); |
- app->Set("launch", app_launch); |
- manifest.Set("app", app); |
+ app->Set("launch", std::move(app_launch)); |
+ manifest.Set("app", std::move(app)); |
break; |
} |