Add DBusStatistics and DBusLogSource to log and show dbus stats

chromeos/dbus/debug_daemon_client.cc

 // 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.
 
 #include <fcntl.h>
 #include <unistd.h>
 
 #include "chromeos/dbus/debug_daemon_client.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/chromeos/chromeos_version.h"
 #include "base/eintr_wrapper.h"
 #include "base/memory/ref_counted_memory.h"
+#include "base/message_loop.h"
 #include "base/platform_file.h"
 #include "base/string_util.h"
 #include "base/threading/worker_pool.h"
 #include "dbus/bus.h"
 #include "dbus/message.h"
 #include "dbus/object_path.h"
 #include "dbus/object_proxy.h"
 #include "net/base/file_stream.h"
 #include "net/base/io_buffer.h"
 #include "net/base/net_errors.h"
@@ skipping 505 matching lines @@
   virtual void StartSystemTracing() OVERRIDE {}
   virtual bool RequestStopSystemTracing(const StopSystemTracingCallback&
       callback) OVERRIDE {
     std::string no_data;
     callback.Run(base::RefCountedString::TakeString(&no_data));
     return true;
   }
   virtual void GetRoutes(bool numeric, bool ipv6,
                          const GetRoutesCallback& callback) OVERRIDE {
     std::vector<std::string> empty;
-    callback.Run(false, empty);
+    MessageLoop::current()->PostTask(FROM_HERE,
+                                     base::Bind(callback, false, empty));
   }
   virtual void GetNetworkStatus(const GetNetworkStatusCallback& callback)
       OVERRIDE {
-    callback.Run(false, "");
+    MessageLoop::current()->PostTask(FROM_HERE,
+                                     base::Bind(callback, false, ""));
   }
   virtual void GetModemStatus(const GetModemStatusCallback& callback)
       OVERRIDE {
-    callback.Run(false, "");
+    MessageLoop::current()->PostTask(FROM_HERE,
+                                     base::Bind(callback, false, ""));
   }
   virtual void GetNetworkInterfaces(
       const GetNetworkInterfacesCallback& callback) OVERRIDE {
-    callback.Run(false, "");
+    MessageLoop::current()->PostTask(FROM_HERE,
+                                     base::Bind(callback, false, ""));
   }
   virtual void GetAllLogs(const GetLogsCallback& callback) OVERRIDE {
     std::map<std::string, std::string> empty;
-    callback.Run(false, empty);
+    MessageLoop::current()->PostTask(FROM_HERE,
+                                     base::Bind(callback, false, empty));
   }
   virtual void GetUserLogFiles(const GetLogsCallback& callback) OVERRIDE {
     std::map<std::string, std::string> empty;
-    callback.Run(false, empty);
+    MessageLoop::current()->PostTask(FROM_HERE,
+                                     base::Bind(callback, false, empty));
   }
 
   virtual void TestICMP(const std::string& ip_address,
                         const TestICMPCallback& callback) OVERRIDE {
-    callback.Run(false, "");
+    MessageLoop::current()->PostTask(FROM_HERE,
+                                     base::Bind(callback, false, ""));

[satorux1, 2012/11/12 00:40:43]

Changes in this file seem to be unrelated to the main purpose of the patch.
Maybe create a separate patch?

[stevenjb, 2012/11/12 19:46:40]

Sorry, meant to mention that. These were from
https://chromiumcodereview.appspot.com/11364169/, now in ToT.

   }
 };
 
 DebugDaemonClient::DebugDaemonClient() {
 }
 
 DebugDaemonClient::~DebugDaemonClient() {
 }
 
 // static
 DebugDaemonClient::StopSystemTracingCallback
 DebugDaemonClient::EmptyStopSystemTracingCallback() {
   return base::Bind(&EmptyStopSystemTracingCallbackBody);
 }
 
 // static
 DebugDaemonClient* DebugDaemonClient::Create(DBusClientImplementationType type,
                                    dbus::Bus* bus) {
   if (type == REAL_DBUS_CLIENT_IMPLEMENTATION)
     return new DebugDaemonClientImpl(bus);
   DCHECK_EQ(STUB_DBUS_CLIENT_IMPLEMENTATION, type);
   return new DebugDaemonClientStubImpl();
 }
 
 }  // namespace chromeos