Replace unique_ptr.reset/release with std::move under src/extensions

BUG=644626

Review-Url: https://codereview.chromium.org/2551723002
Cr-Commit-Position: refs/heads/master@{#437542}

extensions/browser/api/alarms/alarm_manager.cc

@@ -309,7 +309,7 @@ void AlarmManager::WriteToStorage(const std::string& extension_id) {
   if (list != alarms_.end())
     alarms = AlarmsToValue(list->second);
   else
-    alarms.reset(AlarmsToValue(AlarmList()).release());
+    alarms = AlarmsToValue(AlarmList());
   storage->SetExtensionValue(extension_id, kRegisteredAlarms,
                              std::move(alarms));
 }

extensions/browser/api/cast_channel/cast_message_util.cc

@@ -99,7 +99,7 @@ bool CastMessageToMessageInfo(const CastMessage& message_proto,
   }
   if (value.get()) {
     DCHECK(!message->data.get());
-    message->data.reset(value.release());
+    message->data = std::move(value);
     return true;
   } else {
     return false;

extensions/browser/api/networking_private/networking_private_delegate.h

@@ -104,7 +104,7 @@ class NetworkingPrivateDelegate : public KeyedService {
   ~NetworkingPrivateDelegate() override;
 
   void set_ui_delegate(std::unique_ptr<UIDelegate> ui_delegate) {
-    ui_delegate_.reset(ui_delegate.release());
+    ui_delegate_ = std::move(ui_delegate);
   }
 
   const UIDelegate* ui_delegate() { return ui_delegate_.get(); }

extensions/browser/api/networking_private/networking_private_delegate_factory.cc

@@ -59,12 +59,12 @@ NetworkingPrivateDelegateFactory::~NetworkingPrivateDelegateFactory() {
 
 void NetworkingPrivateDelegateFactory::SetVerifyDelegateFactory(
     std::unique_ptr<VerifyDelegateFactory> factory) {
-  verify_factory_.reset(factory.release());
+  verify_factory_ = std::move(factory);
 }
 
 void NetworkingPrivateDelegateFactory::SetUIDelegateFactory(
     std::unique_ptr<UIDelegateFactory> factory) {
-  ui_factory_.reset(factory.release());
+  ui_factory_ = std::move(factory);
 }
 
 KeyedService* NetworkingPrivateDelegateFactory::BuildServiceInstanceFor(

extensions/browser/api/web_request/web_request_api.cc

@@ -2299,9 +2299,9 @@ WebRequestInternalEventHandledFunction::Run() {
           response_headers->push_back(helpers::ResponseHeader(name, value));
       }
       if (has_request_headers)
-        response->request_headers.reset(request_headers.release());
+        response->request_headers = std::move(request_headers);
       else
-        response->response_headers.reset(response_headers.release());
+        response->response_headers = std::move(response_headers);
     }
 
     if (value->HasKey(keys::kAuthCredentialsKey)) {

extensions/browser/updater/request_queue_impl.h

@@ -112,8 +112,8 @@ void RequestQueue<T>::StartNextRequest() {
   std::pop_heap(
       pending_requests_.begin(), pending_requests_.end(), CompareRequests);
 
-  active_backoff_entry_.reset(pending_requests_.back().backoff_entry.release());
-  active_request_.reset(pending_requests_.back().request.release());
+  active_backoff_entry_ = std::move(pending_requests_.back().backoff_entry);
+  active_request_ = std::move(pending_requests_.back().request);
 
   pending_requests_.pop_back();
 

extensions/browser/user_script_loader.cc

@@ -359,7 +359,7 @@ void UserScriptLoader::SetReady(bool ready) {
 void UserScriptLoader::OnScriptsLoaded(
     std::unique_ptr<UserScriptList> user_scripts,
     std::unique_ptr<base::SharedMemory> shared_memory) {
-  user_scripts_.reset(user_scripts.release());
+  user_scripts_ = std::move(user_scripts);
   if (pending_load_) {
     // While we were loading, there were further changes. Don't bother
     // notifying about these scripts and instead just immediately reload.
@@ -381,7 +381,7 @@ void UserScriptLoader::OnScriptsLoaded(
   }
 
   // We've got scripts ready to go.
-  shared_memory_.reset(shared_memory.release());
+  shared_memory_ = std::move(shared_memory);
 
   for (content::RenderProcessHost::iterator i(
            content::RenderProcessHost::AllHostsIterator());