[Mojo Video Capture] Stop service when last client disconnects.

services/video_capture/service_manager_service_impl.h

+// Copyright 2016 The Chromium Authors. All rights reserved.

[mcasas, 2017/04/26 22:12:31]

s/2016/2017/

[chfremer, 2017/04/26 23:33:36]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef SERVICES_VIDEO_CAPTURE_SERVICE_MANAGER_SERVICE_IMPL_H_
+#define SERVICES_VIDEO_CAPTURE_SERVICE_MANAGER_SERVICE_IMPL_H_
+
+#include <memory>
+
+#include "mojo/public/cpp/bindings/binding_set.h"
+#include "services/service_manager/public/cpp/binder_registry.h"
+#include "services/service_manager/public/cpp/interface_factory.h"
+#include "services/service_manager/public/cpp/service.h"
+#include "services/service_manager/public/cpp/service_context_ref.h"
+#include "services/video_capture/public/interfaces/service.mojom.h"
+
+namespace video_capture {
+
+class ServiceManagerServiceImpl : public service_manager::Service {

[Ken Rockot(use gerrit already), 2017/04/26 19:40:19]

nit: Hmm, is this rename really necessary?

[chfremer, 2017/04/26 20:01:54]

Hmm, not absolutely necessary, but I feel it is useful to disambiguate from
VideoCaptureServiceImpl, since both classes would equally qualify for being
named just ServiceImpl.

[mcasas, 2017/04/26 22:12:31]

namespace disambiguates, right?
video_capture::Servicelmpl sounds quite clear to me.

[chfremer, 2017/04/26 23:33:36]

Yes, to me too, but unfortunately both classes live in the video_capture
namespace.

I think Ken's suggestion to avoid using a top-level Mojo interface named
"Service" is good.
I'll go an rename mojom.Service -> mojom.DeviceFactoryProvider and then the rest
follows.

Done.

+ public:
+  ServiceManagerServiceImpl();
+  ~ServiceManagerServiceImpl() override;
+
+  // service_manager::Service:

[mcasas, 2017/04/26 22:12:31]

// service_manager::Service implementation.

[chfremer, 2017/04/26 23:33:36]

Done.

+  void OnStart() override;
+  void OnBindInterface(const service_manager::ServiceInfo& source_info,
+                       const std::string& interface_name,
+                       mojo::ScopedMessagePipeHandle interface_pipe) override;
+  bool OnServiceManagerConnectionLost() override;
+
+ private:
+  void OnVideoCaptureServiceRequest(mojom::ServiceRequest request);
+  void SetShutdownDelayInSeconds(float seconds);
+  void MaybeRequestQuitDelayed();
+  void MaybeRequestQuit();
+
+  float shutdown_delay_in_seconds_;
+  service_manager::BinderRegistry registry_;
+  std::unique_ptr<service_manager::ServiceContextRefFactory> ref_factory_;
+  base::WeakPtrFactory<ServiceManagerServiceImpl> weak_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(ServiceManagerServiceImpl);
+};
+
+}  // namespace video_capture
+
+#endif  // SERVICES_VIDEO_CAPTURE_SERVICE_MANAGER_SERVICE_IMPL_H_