Index: chrome/browser/extensions/extension_message_bubble_controller_unittest.cc |
diff --git a/chrome/browser/extensions/extension_message_bubble_controller_unittest.cc b/chrome/browser/extensions/extension_message_bubble_controller_unittest.cc |
index 9cae7a2d720453d53b322b185b5408772405a363..118174890c9f4cc085a918216eaed3d1b891710d 100644 |
--- a/chrome/browser/extensions/extension_message_bubble_controller_unittest.cc |
+++ b/chrome/browser/extensions/extension_message_bubble_controller_unittest.cc |
@@ -9,6 +9,7 @@ |
#include "base/bind_helpers.h" |
#include "base/command_line.h" |
#include "base/macros.h" |
+#include "base/memory/ptr_util.h" |
#include "base/strings/string_number_conversions.h" |
#include "base/strings/string_util.h" |
#include "base/strings/utf_string_conversions.h" |
@@ -46,9 +47,9 @@ const char kId1[] = "iccfkkhkfiphcjdakkmcjmkfboccmndk"; |
const char kId2[] = "ajjhifimiemdpmophmkkkcijegphclbl"; |
const char kId3[] = "ioibbbfddncmmabjmpokikkeiofalaek"; |
-scoped_ptr<KeyedService> BuildOverrideRegistrar( |
+std::unique_ptr<KeyedService> BuildOverrideRegistrar( |
content::BrowserContext* context) { |
- return make_scoped_ptr( |
+ return base::WrapUnique( |
new extensions::ExtensionWebUIOverrideRegistrar(context)); |
} |
@@ -330,7 +331,7 @@ class ExtensionMessageBubbleTest : public BrowserWithTestWindowTest { |
Manifest::Location location, |
const std::string& data, |
const std::string& id) { |
- scoped_ptr<base::DictionaryValue> parsed_manifest( |
+ std::unique_ptr<base::DictionaryValue> parsed_manifest( |
api_test_utils::ParseDictionary(data)); |
return api_test_utils::CreateExtension(location, parsed_manifest.get(), id); |
} |
@@ -338,7 +339,7 @@ class ExtensionMessageBubbleTest : public BrowserWithTestWindowTest { |
ExtensionService* service_; |
private: |
- scoped_ptr<base::CommandLine> command_line_; |
+ std::unique_ptr<base::CommandLine> command_line_; |
DISALLOW_COPY_AND_ASSIGN(ExtensionMessageBubbleTest); |
}; |
@@ -348,7 +349,7 @@ TEST_F(ExtensionMessageBubbleTest, BubbleReshowsOnDeactivationDismissal) { |
ASSERT_TRUE(LoadExtensionOverridingNtp("1", kId1, Manifest::INTERNAL)); |
ASSERT_TRUE(LoadExtensionOverridingNtp("2", kId2, Manifest::INTERNAL)); |
- scoped_ptr<TestExtensionMessageBubbleController> controller( |
+ std::unique_ptr<TestExtensionMessageBubbleController> controller( |
new TestExtensionMessageBubbleController( |
new NtpOverriddenBubbleDelegate(browser()->profile()), browser())); |
@@ -417,7 +418,7 @@ TEST_F(ExtensionMessageBubbleTest, WipeoutControllerTest) { |
ASSERT_TRUE(LoadGenericExtension("2", kId2, Manifest::UNPACKED)); |
ASSERT_TRUE(LoadGenericExtension("3", kId3, Manifest::EXTERNAL_POLICY)); |
- scoped_ptr<TestExtensionMessageBubbleController> controller( |
+ std::unique_ptr<TestExtensionMessageBubbleController> controller( |
new TestExtensionMessageBubbleController( |
new SuspiciousExtensionBubbleDelegate(browser()->profile()), |
browser())); |
@@ -496,10 +497,9 @@ TEST_F(ExtensionMessageBubbleTest, DevModeControllerTest) { |
ASSERT_TRUE(LoadGenericExtension("2", kId2, Manifest::UNPACKED)); |
ASSERT_TRUE(LoadGenericExtension("3", kId3, Manifest::EXTERNAL_POLICY)); |
- scoped_ptr<TestExtensionMessageBubbleController> controller( |
+ std::unique_ptr<TestExtensionMessageBubbleController> controller( |
new TestExtensionMessageBubbleController( |
- new DevModeBubbleDelegate(browser()->profile()), |
- browser())); |
+ new DevModeBubbleDelegate(browser()->profile()), browser())); |
// The list will contain one enabled unpacked extension. |
EXPECT_TRUE(controller->ShouldShow()); |
@@ -630,7 +630,7 @@ TEST_F(ExtensionMessageBubbleTest, MAYBE_SettingsApiControllerTest) { |
} |
SettingsApiOverrideType type = static_cast<SettingsApiOverrideType>(i); |
- scoped_ptr<TestExtensionMessageBubbleController> controller( |
+ std::unique_ptr<TestExtensionMessageBubbleController> controller( |
new TestExtensionMessageBubbleController( |
new SettingsApiBubbleDelegate(browser()->profile(), type), |
browser())); |
@@ -741,10 +741,9 @@ TEST_F(ExtensionMessageBubbleTest, NtpOverriddenControllerTest) { |
ASSERT_TRUE(LoadExtensionOverridingNtp("2", kId2, Manifest::UNPACKED)); |
ASSERT_TRUE(LoadExtensionOverridingStart("3", kId3, Manifest::UNPACKED)); |
- scoped_ptr<TestExtensionMessageBubbleController> controller( |
+ std::unique_ptr<TestExtensionMessageBubbleController> controller( |
new TestExtensionMessageBubbleController( |
- new NtpOverriddenBubbleDelegate(browser()->profile()), |
- browser())); |
+ new NtpOverriddenBubbleDelegate(browser()->profile()), browser())); |
// The list will contain one enabled unpacked extension (ext 2). |
EXPECT_TRUE(controller->ShouldShow()); |
@@ -877,10 +876,9 @@ TEST_F(ExtensionMessageBubbleTest, MAYBE_ProxyOverriddenControllerTest) { |
SetInstallTime(kId2, base::Time::Now(), prefs); |
SetInstallTime(kId3, old_enough, prefs); |
- scoped_ptr<TestExtensionMessageBubbleController> controller( |
+ std::unique_ptr<TestExtensionMessageBubbleController> controller( |
new TestExtensionMessageBubbleController( |
- new ProxyOverriddenBubbleDelegate(browser()->profile()), |
- browser())); |
+ new ProxyOverriddenBubbleDelegate(browser()->profile()), browser())); |
// The second extension is too new to warn about. |
EXPECT_FALSE(controller->ShouldShow()); |