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..73493052695120a2f43230fa484edc55ea879ab0 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,10 @@ class CHROMEOS_EXPORT UpstartClient : public DBusClient {
   // Restarts authpolicyd.
   virtual void RestartAuthPolicyService() = 0;
 
+  // Process management of a separate media analytics process.

[tbarzic, 2017/05/11 16:55:55]

Comment each of the methods:

Starts media analytics process.

Stops media analytics process.

[Luke Sorenson, 2017/05/11 18:16:48]

Done.

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