IPC: Remove unnecessary conversions of BrokerableAttachment.

ipc/ipc_message_attachment_set.h

 // Copyright (c) 2011 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_IPC_MESSAGE_ATTACHMENT_SET_H_
 #define IPC_IPC_MESSAGE_ATTACHMENT_SET_H_
 
 #include <vector>
 
 #include "base/macros.h"
@@ skipping 69 matching lines @@
   // Similar to GetNonBrokerableAttachmentAt, but there are no ordering
   // requirements.
   scoped_refptr<MessageAttachment> GetBrokerableAttachmentAt(unsigned index);
 
   // This must be called after transmitting the descriptors returned by
   // PeekDescriptors. It marks all the non-brokerable descriptors as consumed
   // and closes those which are auto-close.
   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);
 
 #if defined(OS_POSIX)
   // This is the maximum number of descriptors per message. We need to know this
   // because the control message kernel interface has to be given a buffer which
   // is large enough to store all the descriptor numbers. Otherwise the kernel
@@ skipping 49 matching lines @@
   // all the descriptors have been read (with GetNthDescriptor). Secondly, we
   // can check that they are read in order.
   mutable unsigned consumed_descriptor_highwater_;
 
   DISALLOW_COPY_AND_ASSIGN(MessageAttachmentSet);
 };
 
 }  // namespace IPC
 
 #endif  // IPC_IPC_MESSAGE_ATTACHMENT_SET_H_