| Index: chrome/common/extensions/manifest_tests/extension_manifests_platformapp_unittest.cc
|
| diff --git a/chrome/common/extensions/manifest_tests/extension_manifests_platformapp_unittest.cc b/chrome/common/extensions/manifest_tests/extension_manifests_platformapp_unittest.cc
|
| index 53ab8623c0db4f2431266261af39aed566c8fb57..06ccafb23bde94c987b47ce42b9374090af4e97d 100644
|
| --- a/chrome/common/extensions/manifest_tests/extension_manifests_platformapp_unittest.cc
|
| +++ b/chrome/common/extensions/manifest_tests/extension_manifests_platformapp_unittest.cc
|
| @@ -124,13 +124,12 @@ TEST_F(PlatformAppsManifestTest, CertainApisRequirePlatformApps) {
|
| std::vector<std::unique_ptr<ManifestData>> manifests;
|
| // Create each manifest.
|
| for (const char* api_name : kPlatformAppExperimentalApis) {
|
| - // DictionaryValue will take ownership of this ListValue.
|
| - base::ListValue *permissions = new base::ListValue();
|
| + auto permissions = base::MakeUnique<base::ListValue>();
|
| permissions->AppendString("experimental");
|
| permissions->AppendString(api_name);
|
| - manifest->Set("permissions", permissions);
|
| - manifests.push_back(
|
| - base::MakeUnique<ManifestData>(manifest->CreateDeepCopy(), ""));
|
| + manifest->Set("permissions", std::move(permissions));
|
| + manifests.push_back(base::MakeUnique<ManifestData>(
|
| + base::MakeUnique<base::DictionaryValue>(*manifest), ""));
|
| }
|
| // First try to load without any flags. This should warn for every API.
|
| for (const std::unique_ptr<ManifestData>& manifest : manifests) {
|
|
|