[Tracing Clock Sync] Add TracingAgent interface in Chrome

chromeos/dbus/fake_debug_daemon_client.cc

 // Copyright 2013 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 "chromeos/dbus/fake_debug_daemon_client.h"
 
 #include <map>
 #include <string>
 
 #include "base/bind.h"
@@ skipping 20 matching lines @@
     base::File file,
     scoped_refptr<base::TaskRunner> task_runner,
     const GetDebugLogsCallback& callback) {
   callback.Run(true);
 }
 
 void FakeDebugDaemonClient::SetDebugMode(const std::string& subsystem,
                                          const SetDebugModeCallback& callback) {
   callback.Run(false);
 }
-void FakeDebugDaemonClient::StartSystemTracing() {}
 
-bool FakeDebugDaemonClient::RequestStopSystemTracing(
-    scoped_refptr<base::TaskRunner> task_runner,
-    const StopSystemTracingCallback& callback) {
+std::string FakeDebugDaemonClient::GetTracingAgentName() {
+  return kCrOSTracingAgentName;
+}
+
+bool FakeDebugDaemonClient::StartAgentTracing(
+    const base::trace_event::TraceConfig& trace_config) {
+  return true;
+}
+
+void FakeDebugDaemonClient::StopAgentTracing(
+    const StopAgentTracingCallback& callback) {
   std::string no_data;
   callback.Run(base::RefCountedString::TakeString(&no_data));
-  return true;
 }
 
+bool FakeDebugDaemonClient::SupportsExplicitClockSync() {
+  return false;
+}
+
+void FakeDebugDaemonClient::RecordClockSyncMarker(
+    scoped_ptr<base::DictionaryValue> marker) {
+  DCHECK(SupportsExplicitClockSync());
+}
+
+void FakeDebugDaemonClient::IssueClockSyncMarker() {

[oystein (OOO til 10th of July), 2015/11/24 22:52:10]

The pattern of these three above functions seem to be the default, and repeated
a lot. Maybe this should be the default behavior in the base class, to reduce
the boilerplate?

[Zhen Wang, 2015/11/25 04:24:55]

Done.

+  DCHECK(SupportsExplicitClockSync());
+}
+
+void FakeDebugDaemonClient::SetStopAgentTracingTaskRunner(
+    scoped_refptr<base::TaskRunner> task_runner) {}
+
 void FakeDebugDaemonClient::GetRoutes(bool numeric,
                                       bool ipv6,
                                       const GetRoutesCallback& callback) {
   std::vector<std::string> empty;
   base::ThreadTaskRunnerHandle::Get()->PostTask(
       FROM_HERE, base::Bind(callback, false, empty));
 }
 
 void FakeDebugDaemonClient::GetNetworkStatus(
     const GetNetworkStatusCallback& callback) {
@@ skipping 114 matching lines @@
   if (!is_available)
     return;
 
   std::vector<WaitForServiceToBeAvailableCallback> callbacks;
   callbacks.swap(pending_wait_for_service_to_be_available_callbacks_);
   for (size_t i = 0; i < callbacks.size(); ++i)
     callbacks[i].Run(is_available);
 }
 
 }  // namespace chromeos