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

chrome/browser/extensions/api/copresence/copresence_translations.cc

 // Copyright 2014 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/copresence/copresence_translations.h"
 
 #include "chrome/common/extensions/api/copresence.h"
 #include "components/copresence/proto/data.pb.h"
 #include "components/copresence/proto/enums.pb.h"
 #include "components/copresence/proto/rpcs.pb.h"
@@ skipping 179 matching lines @@
     apps_by_subscription_id->erase(subscription);
   }
 
   request->mutable_manage_subscriptions_request()->add_id_to_unsubscribe(
       subscription_id);
   DVLOG(2) << "Cancelling subscription \"" << subscription_id << "\" for app \""
            << app_id << "\"";
   return true;
 }
 
-bool PrepareReportRequestProto(
-    const std::vector<linked_ptr<Operation>>& operations,
-    const std::string& app_id,
-    SubscriptionToAppMap* apps_by_subscription_id,
-    ReportRequest* request) {
-  for (const linked_ptr<Operation>& op : operations) {
-    DCHECK(op.get());
-
+bool PrepareReportRequestProto(const std::vector<Operation>& operations,
+                               const std::string& app_id,
+                               SubscriptionToAppMap* apps_by_subscription_id,
+                               ReportRequest* request) {
+  for (const Operation& op : operations) {
     // Verify our object has exactly one operation.
-    if (static_cast<int>(op->publish != nullptr) +
-        static_cast<int>(op->subscribe != nullptr) +
-        static_cast<int>(op->unpublish != nullptr) +
-        static_cast<int>(op->unsubscribe != nullptr) != 1) {
+    if (static_cast<int>(op.publish != nullptr) +
+            static_cast<int>(op.subscribe != nullptr) +
+            static_cast<int>(op.unpublish != nullptr) +
+            static_cast<int>(op.unsubscribe != nullptr) !=
+        1) {
       return false;
     }
 
-    if (op->publish) {
-      if (!AddPublishToRequest(app_id, *(op->publish), request))
+    if (op.publish) {
+      if (!AddPublishToRequest(app_id, *(op.publish), request))
         return false;
-    } else if (op->subscribe) {
-      if (!AddSubscribeToRequest(
-              app_id, *(op->subscribe), apps_by_subscription_id, request))
+    } else if (op.subscribe) {
+      if (!AddSubscribeToRequest(app_id, *(op.subscribe),
+                                 apps_by_subscription_id, request))
         return false;
-    } else if (op->unpublish) {
-      if (!AddUnpublishToRequest(op->unpublish->unpublish_id, request))
+    } else if (op.unpublish) {
+      if (!AddUnpublishToRequest(op.unpublish->unpublish_id, request))
         return false;
-    } else {  // if (op->unsubscribe)
-      if (!AddUnsubscribeToRequest(app_id,
-                                   op->unsubscribe->unsubscribe_id,
-                                   apps_by_subscription_id,
-                                   request))
+    } else {  // if (op.unsubscribe)
+      if (!AddUnsubscribeToRequest(app_id, op.unsubscribe->unsubscribe_id,
+                                   apps_by_subscription_id, request))
         return false;
     }
   }
 
   return true;
 }
 
 }  // namespace extensions