DBus MediaAnalyticsClient and media_perception pb.

chromeos/dbus/fake_upstart_client.cc

 // Copyright 2016 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_upstart_client.h"
 
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/fake_auth_policy_client.h"
 
 namespace chromeos {
@@ skipping 11 matching lines @@
 }
 
 void FakeUpstartClient::RestartAuthPolicyService() {
   FakeAuthPolicyClient* authpolicy_client = static_cast<FakeAuthPolicyClient*>(
       DBusThreadManager::Get()->GetAuthPolicyClient());
   DLOG_IF(WARNING, !authpolicy_client->started())
       << "Trying to restart authpolicyd which is not started";
   authpolicy_client->set_started(true);
 }
 
+void FakeUpstartClient::StartMediaAnalytics(const UpstartCallback& callback) {

[tbarzic, 2017/05/09 20:28:41]

it might be useful to tailor FakeMediaAnalytics client based on this -  e.g.
make all its methods fail until this is called.

[Luke Sorenson, 2017/05/09 22:21:25]

Acknowledged. For now, going to keep them independent if that's okay.

[tbarzic, 2017/05/10 03:26:24]

I'd prefer it done now, given that it would match real device behavior more
exactly; and to keep it consistent with the fake behavior for authpolicy client.

[Luke Sorenson, 2017/05/10 16:38:22]

Done.

+  callback.Run(true);
+}
+
+void FakeUpstartClient::StopMediaAnalytics() {}
+
 }  // namespace chromeos