IPC: Remove unnecessary conversions of BrokerableAttachment.

ipc/ipc_channel_reader_unittest.cc

 // 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.
 
 #include "build/build_config.h"
 
 #include <limits>
 #include <set>
 
 #include "ipc/attachment_broker.h"
@@ skipping 24 matching lines @@
   BrokerableType GetBrokerableType() const override { return WIN_HANDLE; }
 
  private:
   ~MockAttachment() override {}
 };
 
 class MockAttachmentBroker : public AttachmentBroker {
  public:
   typedef std::set<scoped_refptr<BrokerableAttachment>> AttachmentSet;
 
-  bool SendAttachmentToProcess(BrokerableAttachment* attachment,
-                               base::ProcessId destination_process) override {
+  bool SendAttachmentToProcess(
+      const scoped_refptr<BrokerableAttachment>& attachment,
+      base::ProcessId destination_process) override {
     return false;
   }
 
   bool OnMessageReceived(const Message& message) override { return false; }
 
   void AddAttachment(scoped_refptr<BrokerableAttachment> attachment) {
     get_attachments()->push_back(attachment);
     NotifyObservers(attachment->GetIdentifier());
   }
 };
@@ skipping 129 matching lines @@
   header.payload_size = std::numeric_limits<int32_t>::max();
   EXPECT_FALSE(reader.TranslateInputData(
       reinterpret_cast<const char*>(&header), sizeof(header)));
   EXPECT_LE(reader.input_overflow_buf_.capacity(), capacity_before);
 }
 
 #endif  // !USE_ATTACHMENT_BROKER
 
 }  // namespace internal
 }  // namespace IPC