Some linked_ptr -> unique_ptr conversion in extensions/browser.

chrome/browser/extensions/extension_service.cc

Index: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index 9f425f57445b17dc110c3530babf849ffaea4c11..819d88bb7f4fde70923d947070f3282065970692 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -176,8 +176,7 @@ void ExtensionService::ClearProvidersForTesting() {
 void ExtensionService::AddProviderForTesting(
     ExternalProviderInterface* test_provider) {

[Devlin, 2016/08/30 19:21:53]

Can we add a TODO to make this take a unique_ptr (or, it looks like it's only
called in three places, if you wanted to do it in this cl)?

[lazyboy, 2016/08/30 20:30:56]

Done.
Changed those places.

I also tried changing AddMockExternalProvider's signature to take unique_ptr in
extensoin_service_unittest.cc, gave up as I thought changing that to just wrap a
raw ptr with WrapUnique isn't doing much. The pattern currently looks like:

MockExtensionProvider* provider = new ..;
AddMockExternalProvider(provider);
..
keep accessing |provider| raw ptr.

[Devlin, 2016/08/30 22:48:28]

Ack; no need to change it in this CL.

   CHECK(test_provider);
-  external_extension_providers_.push_back(
-      linked_ptr<ExternalProviderInterface>(test_provider));
+  external_extension_providers_.push_back(base::WrapUnique(test_provider));
 }
 
 void ExtensionService::BlacklistExtensionForTest(