More graceful shutdown for ArcSession.

components/arc/arc_bridge_service_impl.h

Index: components/arc/arc_bridge_service_impl.h
diff --git a/components/arc/arc_bridge_service_impl.h b/components/arc/arc_bridge_service_impl.h
index 7f92c68c18ae6f14224856a0e378d423fab4f63d..9b4dfa71699e09328d8ff6dc4040546b5a12891d 100644
--- a/components/arc/arc_bridge_service_impl.h
+++ b/components/arc/arc_bridge_service_impl.h
@@ -12,6 +12,8 @@
 #include "base/files/scoped_file.h"
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "base/task_runner.h"
 #include "components/arc/arc_bridge_service.h"
 #include "components/arc/arc_session.h"
 #include "mojo/public/cpp/bindings/binding.h"
@@ -31,12 +33,14 @@ class ArcBridgeServiceImpl : public ArcBridgeService,
   // for testing purpose.
   using ArcSessionFactory = base::Callback<std::unique_ptr<ArcSession>()>;
 
-  ArcBridgeServiceImpl();
+  explicit ArcBridgeServiceImpl(
+      scoped_refptr<base::TaskRunner> blocking_task_runner);

[Luis Héctor Chávez, 2016/10/21 00:43:41]

nit: const scoped_refptr<...>& to avoid intermediate copies, since it is
copiable.

[hidehiko, 2016/10/21 04:34:03]

Done.

   ~ArcBridgeServiceImpl() override;
 
-  void HandleStartup() override;
-
-  void Shutdown() override;
+  // ArcBridgeService overrides:
+  void RequestStart() override;
+  void RequestStop() override;
+  void OnShutdown() override;
 
   // Inject a factory to create ArcSession instance for testing purpose.
   // |factory| must not be null.