Split LivenessService out from LibCrosService.

chromeos/dbus/services/liveness_service_provider.h

Index: chromeos/dbus/services/liveness_service_provider.h
diff --git a/chromeos/dbus/services/liveness_service_provider.h b/chromeos/dbus/services/liveness_service_provider.h
index 75092a3d73d79a1e6b1126a7c4a1c4a03179c8c9..be4028db115f73702f86ae43591ffab8734edf7d 100644
--- a/chromeos/dbus/services/liveness_service_provider.h
+++ b/chromeos/dbus/services/liveness_service_provider.h
@@ -26,9 +26,9 @@ namespace chromeos {
 // messages.  It can be tested with the following command:
 //
 // % dbus-send --system --type=method_call --print-reply
-//     --dest=org.chromium.LibCrosService
-//     /org/chromium/LibCrosService
-//     org.chromium.LibCrosServiceInterface.CheckLiveness
+//     --dest=org.chromium.LivenessService
+//     /org/chromium/LivenessService
+//     org.chromium.LivenessServiceInterface.CheckLiveness
 //
 // -> method return sender=:1.9 -> dest=:1.27 reply_serial=2
 //
@@ -36,7 +36,7 @@ namespace chromeos {
 class CHROMEOS_EXPORT LivenessServiceProvider
     : public CrosDBusService::ServiceProviderInterface {
  public:
-  LivenessServiceProvider();
+  explicit LivenessServiceProvider(const char* service_interface);

[Daniel Erat, 2017/03/11 01:27:42]

const std::string& ?

i'd also add a TODO to remove this once we're not creating two of them.

[teravest, 2017/03/13 16:02:28]

Done.

   ~LivenessServiceProvider() override;
 
   // CrosDBusService::ServiceProviderInterface overrides:
@@ -53,6 +53,8 @@ class CHROMEOS_EXPORT LivenessServiceProvider
   void CheckLiveness(dbus::MethodCall* method_call,
                      dbus::ExportedObject::ResponseSender response_sender);
 
+  const char* service_interface_;

[Daniel Erat, 2017/03/11 01:27:42]

std::string?

[teravest, 2017/03/13 16:02:28]

Done.

+
   // Keep this last so that all weak pointers will be invalidated at the
   // beginning of destruction.
   base::WeakPtrFactory<LivenessServiceProvider> weak_ptr_factory_;