[Extensions] Convert APIs to use movable types [1]

chrome/browser/extensions/api/activity_log_private/activity_log_private_api.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/api/activity_log_private/activity_log_private_api.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <utility>
 
@@ skipping 70 matching lines @@
 void ActivityLogAPI::OnListenerAdded(const EventListenerInfo& details) {
   // TODO(felt): Only observe activity_log_ events when we have a customer.
 }
 
 void ActivityLogAPI::OnListenerRemoved(const EventListenerInfo& details) {
   // TODO(felt): Only observe activity_log_ events when we have a customer.
 }
 
 void ActivityLogAPI::OnExtensionActivity(scoped_refptr<Action> activity) {
   scoped_ptr<base::ListValue> value(new base::ListValue());
-  scoped_ptr<ExtensionActivity> activity_arg =
-      activity->ConvertToExtensionActivity();
-  value->Append(activity_arg->ToValue().release());
+  ExtensionActivity activity_arg = activity->ConvertToExtensionActivity();
+  value->Append(activity_arg.ToValue());
   scoped_ptr<Event> event(new Event(
       events::ACTIVITY_LOG_PRIVATE_ON_EXTENSION_ACTIVITY,
       activity_log_private::OnExtensionActivity::kEventName, std::move(value)));
   event->restrict_to_browser_context = browser_context_;
   EventRouter::Get(browser_context_)->BroadcastEvent(std::move(event));
 }
 
 bool ActivityLogPrivateGetExtensionActivitiesFunction::RunAsync() {
   scoped_ptr<activity_log_private::GetExtensionActivities::Params> params(
       activity_log_private::GetExtensionActivities::Params::Create(*args_));
@@ skipping 51 matching lines @@
       base::Bind(
           &ActivityLogPrivateGetExtensionActivitiesFunction::OnLookupCompleted,
           this));
 
   return true;
 }
 
 void ActivityLogPrivateGetExtensionActivitiesFunction::OnLookupCompleted(
     scoped_ptr<std::vector<scoped_refptr<Action> > > activities) {
   // Convert Actions to ExtensionActivities.
-  std::vector<linked_ptr<ExtensionActivity> > result_arr;
-  for (std::vector<scoped_refptr<Action> >::iterator it = activities->begin();
-       it != activities->end();
-       ++it) {
-    result_arr.push_back(
-        make_linked_ptr(it->get()->ConvertToExtensionActivity().release()));
-  }
+  std::vector<ExtensionActivity> result_arr;
+  for (const auto& activity : *activities)
+    result_arr.push_back(activity->ConvertToExtensionActivity());
 
   // Populate the return object.
   scoped_ptr<ActivityResultSet> result_set(new ActivityResultSet);
-  result_set->activities = result_arr;
+  result_set->activities = std::move(result_arr);
   results_ = activity_log_private::GetExtensionActivities::Results::Create(
       *result_set);
 
   SendResponse(true);
 }
 
 bool ActivityLogPrivateDeleteActivitiesFunction::RunAsync() {
   scoped_ptr<activity_log_private::DeleteActivities::Params> params(
       activity_log_private::DeleteActivities::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get());
@@ skipping 33 matching lines @@
     gurls.push_back(GURL(*it));
   }
 
   ActivityLog* activity_log = ActivityLog::GetInstance(GetProfile());
   DCHECK(activity_log);
   activity_log->RemoveURLs(gurls);
   return true;
 }
 
 }  // namespace extensions