[Device Service] Move PowerMonitor into the Device Service

services/device/device_service.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 "services/device/device_service.h"
 
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
 #include "base/memory/weak_ptr.h"
 #include "base/threading/thread_task_runner_handle.h"
+#include "device/power_monitor/power_monitor_message_broadcaster.h"
 #include "device/time_zone_monitor/time_zone_monitor.h"
 #include "services/service_manager/public/cpp/connection.h"
 #include "services/service_manager/public/cpp/interface_registry.h"
 
 namespace device {
 
+namespace {
+// Flag to guarantee device service has singleton instance.
+static bool g_service_instance_alive = false;

[blundell, 2017/01/03 15:20:50]

I wouldn't worry about this -- this is essentially just verifying that the
singleton mechanism is working correctly, which seems more the role of e.g.
service manager integration tests.

[leonhsl(Using Gerrit), 2017/01/04 04:56:28]

Done.

+}
+
 std::unique_ptr<service_manager::Service> CreateDeviceService(
     scoped_refptr<base::SingleThreadTaskRunner> file_task_runner) {
   return base::MakeUnique<DeviceService>(std::move(file_task_runner));
 }
 
 DeviceService::DeviceService(
     scoped_refptr<base::SingleThreadTaskRunner> file_task_runner)
-    : file_task_runner_(std::move(file_task_runner)) {}
+    : file_task_runner_(std::move(file_task_runner)) {
+  DCHECK(!g_service_instance_alive);
+  g_service_instance_alive = true;
+}
 
-DeviceService::~DeviceService() {}
+DeviceService::~DeviceService() {
+  g_service_instance_alive = false;
+}
 
 void DeviceService::OnStart() {}
 
 bool DeviceService::OnConnect(const service_manager::ServiceInfo& remote_info,
                               service_manager::InterfaceRegistry* registry) {
+  registry->AddInterface<mojom::PowerMonitor>(this);
   registry->AddInterface<mojom::TimeZoneMonitor>(this);
   return true;
 }
 
 void DeviceService::Create(const service_manager::Identity& remote_identity,
+                           mojom::PowerMonitorRequest request) {
+  PowerMonitorMessageBroadcaster::Create(std::move(request));
+}
+
+void DeviceService::Create(const service_manager::Identity& remote_identity,
                            mojom::TimeZoneMonitorRequest request) {
   if (!time_zone_monitor_)
     time_zone_monitor_ = device::TimeZoneMonitor::Create(file_task_runner_);
   time_zone_monitor_->Bind(std::move(request));
 }
 
 }  // namespace device