IPC: Remove unnecessary conversions of BrokerableAttachment.

ipc/attachment_broker.h

 // Copyright 2015 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.
 
 #ifndef IPC_ATTACHMENT_BROKER_H_
 #define IPC_ATTACHMENT_BROKER_H_
 
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
+#include "base/memory/ref_counted.h"
 #include "base/process/process_handle.h"
 #include "ipc/brokerable_attachment.h"
 #include "ipc/ipc_export.h"
 #include "ipc/ipc_listener.h"
 
 // If the platform has no attachments that need brokering, then it shouldn't
 // compile any code that calls member functions of AttachmentBroker. This
 // prevents symbols only used by AttachmentBroker and its subclasses from
 // making it into the binary.
 #if defined(OS_WIN) || (defined(OS_MACOSX) && !defined(OS_IOS))
@@ skipping 38 matching lines @@
   static void SetGlobal(AttachmentBroker* broker);
   static AttachmentBroker* GetGlobal();
 
   AttachmentBroker();
   ~AttachmentBroker() override;
 
   // Sends |attachment| to |destination_process|. The implementation uses an
   // IPC::Channel to communicate with the broker process. This may be the same
   // IPC::Channel that is requesting the brokering of an attachment.
   // Returns true on success and false otherwise.
-  virtual bool SendAttachmentToProcess(BrokerableAttachment* attachment,
-                                       base::ProcessId destination_process) = 0;
+  virtual bool SendAttachmentToProcess(
+      const scoped_refptr<BrokerableAttachment>& attachment,
+      base::ProcessId destination_process) = 0;
 
   // Returns whether the attachment was available. If the attachment was
   // available, populates the output parameter |attachment|.
   bool GetAttachmentWithId(BrokerableAttachment::AttachmentId id,
                            scoped_refptr<BrokerableAttachment>* attachment);
 
   // Any given observer should only ever add itself once to the observer list.
   void AddObserver(Observer* observer);
   void RemoveObserver(Observer* observer);
 
@@ skipping 34 matching lines @@
   // better performance.
   AttachmentVector attachments_;
 
   std::vector<Observer*> observers_;
   DISALLOW_COPY_AND_ASSIGN(AttachmentBroker);
 };
 
 }  // namespace IPC
 
 #endif  // IPC_ATTACHMENT_BROKER_H_