Chromium Code Reviews| Index: chrome/browser/extensions/extension_override_apitest.cc |
| diff --git a/chrome/browser/extensions/extension_override_apitest.cc b/chrome/browser/extensions/extension_override_apitest.cc |
| index b1c8ae395904a1a6e870402141403d52478d3c31..6f6872cce5fff4902e2a44fa725fb1c0a5ec2fb2 100644 |
| --- a/chrome/browser/extensions/extension_override_apitest.cc |
| +++ b/chrome/browser/extensions/extension_override_apitest.cc |
| @@ -4,6 +4,7 @@ |
| #include <stddef.h> |
|
jdoerrie
2017/04/06 14:25:51
#include <utility>
vabr (Chromium)
2017/04/07 20:40:41
Done.
|
| +#include "base/memory/ptr_util.h" |
| #include "build/build_config.h" |
|
jdoerrie
2017/04/06 14:25:51
#include "base/values.h"
vabr (Chromium)
2017/04/07 20:40:41
Done.
|
| #include "chrome/browser/extensions/extension_apitest.h" |
| #include "chrome/browser/extensions/extension_service.h" |
| @@ -274,7 +275,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionOverrideTest, ShouldCleanUpDuplicateEntries) { |
| // a preferences file without corresponding UnloadExtension() calls. This is |
| // the same as the above test, except for that it is testing the case where |
| // the file already contains dupes when an extension is loaded. |
| - base::ListValue* list = new base::ListValue(); |
| + auto list = base::MakeUnique<base::ListValue>(); |
| for (size_t i = 0; i < 3; ++i) { |
| std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue()); |
| dict->SetString("entry", "http://www.google.com/"); |
| @@ -285,7 +286,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionOverrideTest, ShouldCleanUpDuplicateEntries) { |
| { |
| DictionaryPrefUpdate update(browser()->profile()->GetPrefs(), |
| ExtensionWebUI::kExtensionURLOverrides); |
| - update.Get()->Set("history", list); |
| + update.Get()->Set("history", std::move(list)); |
| } |
| ASSERT_FALSE(CheckHistoryOverridesContainsNoDupes()); |