DBus MediaAnalyticsClient and media_perception pb.

chromeos/dbus/upstart_client.h

 // 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.
 
 #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"
 
 namespace chromeos {
 
 // UpstartClient is used to communicate with the com.ubuntu.Upstart
 // sevice. All methods should be called from the origin thread (UI thread) which
 // initializes the DBusThreadManager instance.
 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();
 
   // Starts authpolicyd.
   virtual void StartAuthPolicyService() = 0;
 
   // 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();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(UpstartClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_UPSTART_CLIENT_H_