DBus MediaAnalyticsClient and media_perception pb.

chromeos/dbus/upstart_client.h

Index: chromeos/dbus/upstart_client.h
diff --git a/chromeos/dbus/upstart_client.h b/chromeos/dbus/upstart_client.h
index 3aff513f6fb2ee7edf75262ec64ad75ff66e53be..e4179596c4f0d0aed78d90709fa5ff84b624d951 100644
--- a/chromeos/dbus/upstart_client.h
+++ b/chromeos/dbus/upstart_client.h
@@ -5,6 +5,8 @@
 #ifndef CHROMEOS_DBUS_UPSTART_CLIENT_H_
 #define CHROMEOS_DBUS_UPSTART_CLIENT_H_
 
+#include "base/callback.h"
+#include "base/macros.h"
 #include "chromeos/chromeos_export.h"
 #include "chromeos/dbus/dbus_client.h"
 
@@ -17,6 +19,7 @@ class CHROMEOS_EXPORT UpstartClient : public DBusClient {
  public:
   ~UpstartClient() override;
 
+  using UpstartCallback = base::Callback<void(bool succeeded)>;
   // Factory function, creates a new instance and returns ownership.
   // For normal usage, access the singleton via DBusThreadManager::Get().
   static UpstartClient* Create();
@@ -27,6 +30,12 @@ class CHROMEOS_EXPORT UpstartClient : public DBusClient {
   // Restarts authpolicyd.
   virtual void RestartAuthPolicyService() = 0;
 
+  // Starts the separate media analytics process.

[tbarzic, 2017/05/11 18:49:57]

nit: it's unclear to what separate refers. I'd just remove it.

[Luke Sorenson, 2017/05/11 22:01:56]

Done.

+  virtual void StartMediaAnalytics(const UpstartCallback& callback) = 0;
+
+  // Stops the media analytics process.
+  virtual void StopMediaAnalytics() = 0;
+
  protected:
   // Create() should be used instead.
   UpstartClient();