Code cleaning: Uses scoped_ptr<> to express ownership rather than writing ownership in comments.

dbus/mock_object_proxy.h

 // Copyright (c) 2012 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 DBUS_MOCK_OBJECT_PROXY_H_
 #define DBUS_MOCK_OBJECT_PROXY_H_
 
 #include <string>
 
+#include "dbus/message.h"
 #include "dbus/object_path.h"
 #include "dbus/object_proxy.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 namespace dbus {
 
 // Mock for ObjectProxy.
 class MockObjectProxy : public ObjectProxy {
  public:
   MockObjectProxy(Bus* bus,
                   const std::string& service_name,
                   const ObjectPath& object_path);
 
-  MOCK_METHOD2(CallMethodAndBlock, Response*(MethodCall* method_call,
-                                             int timeout_ms));
+  MOCK_METHOD2(MockCallMethodAndBlock, Response*(MethodCall* method_call,

[satorux1, 2013/01/30 06:36:58]

I guess this indirection is needed because CallMethodAndBlock() returns a
scoped_ptr<>. Please add some comment about it.

[alias of yukishiino, 2013/01/30 13:13:44]

Done.

+                                                 int timeout_ms));
+  virtual scoped_ptr<Response> CallMethodAndBlock(MethodCall* method_call,
+                                                  int timeout_ms) OVERRIDE {
+    return scoped_ptr<Response>(MockCallMethodAndBlock(method_call,
+                                                       timeout_ms));
+  }
   MOCK_METHOD3(CallMethod, void(MethodCall* method_call,
                                 int timeout_ms,
                                 ResponseCallback callback));
   MOCK_METHOD4(CallMethodWithErrorCallback, void(MethodCall* method_call,
                                                  int timeout_ms,
                                                  ResponseCallback callback,
                                                  ErrorCallback error_callback));
   MOCK_METHOD4(ConnectToSignal,
                void(const std::string& interface_name,
                     const std::string& signal_name,
                     SignalCallback signal_callback,
                     OnConnectedCallback on_connected_callback));
   MOCK_METHOD0(Detach, void());
 
  protected:
   virtual ~MockObjectProxy();
 };
 
 }  // namespace dbus
 
 #endif  // DBUS_MOCK_OBJECT_PROXY_H_