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

dbus/mock_object_proxy.h

Index: dbus/mock_object_proxy.h
diff --git a/dbus/mock_object_proxy.h b/dbus/mock_object_proxy.h
index e820f45a08b97517233b75f5b26a63622f0d914f..358053945659d1c743acf795b90eec854bfa2d8c 100644
--- a/dbus/mock_object_proxy.h
+++ b/dbus/mock_object_proxy.h
@@ -7,6 +7,7 @@
 
 #include <string>
 
+#include "dbus/message.h"
 #include "dbus/object_path.h"
 #include "dbus/object_proxy.h"
 #include "testing/gmock/include/gmock/gmock.h"
@@ -20,8 +21,13 @@ class MockObjectProxy : public ObjectProxy {
                   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));