IPC: Remove unnecessary conversions of BrokerableAttachment.

ipc/ipc_message_attachment_set.h

Index: ipc/ipc_message_attachment_set.h
diff --git a/ipc/ipc_message_attachment_set.h b/ipc/ipc_message_attachment_set.h
index 4276fcd9cdc14d899a1863c5467dc0c5895c4479..572c399e8d1750178722299471f1b0dc04ff04e5 100644
--- a/ipc/ipc_message_attachment_set.h
+++ b/ipc/ipc_message_attachment_set.h
@@ -87,12 +87,13 @@ class IPC_EXPORT MessageAttachmentSet
   void CommitAllDescriptors();
 
   // Returns a vector of all brokerable attachments.
-  std::vector<BrokerableAttachment*> GetBrokerableAttachments() const;
+  std::vector<scoped_refptr<IPC::BrokerableAttachment>>
+  GetBrokerableAttachments() const;
 
   // Replaces a placeholder brokerable attachment with |attachment|, matching
   // them by their id.
   void ReplacePlaceholderWithAttachment(
-      const scoped_refptr<BrokerableAttachment>& attachment);
+      scoped_refptr<BrokerableAttachment> attachment);

[Tom Sepez, 2015/10/29 19:12:51]

and here.

 
 #if defined(OS_POSIX)
   // This is the maximum number of descriptors per message. We need to know this