Refactor ArcBridgeService detection and initialization to a separate class ArcServiceLauncher.

components/arc/arc_bridge_service.h

Index: components/arc/arc_bridge_service.h
diff --git a/components/arc/arc_bridge_service.h b/components/arc/arc_bridge_service.h
index 56179d0287cefcf5a19a29de2272b2342c9bd5ec..fea9b9eedc733b11928be52bab1f7b07fabbb5a6 100644
--- a/components/arc/arc_bridge_service.h
+++ b/components/arc/arc_bridge_service.h
@@ -131,10 +131,10 @@ class ArcBridgeService : public ArcBridgeHost {
   // switch.
   static bool GetEnabled(const base::CommandLine* command_line);
 
-  // DetectAvailability() should be called once D-Bus is available. It will
-  // call CheckArcAvailability() on the session_manager. This can only be
-  // called on the thread that this class was created on.
-  virtual void DetectAvailability() = 0;
+  // SetArcAvailability() should be called once CheckArcAvailability() on the
+  // session_manager is called. This can only be called on the thread that this
+  // class was created on.
+  virtual void SetArcAvailability(bool availability) = 0;

[hidehiko, 2016/02/22 08:08:40]

ArcBridgeService::SetArcAbailability and ArcBridgeService::SetAvailable sounds
confusing. Can we re-sort?

 
   // HandleStartup() should be called upon profile startup.  This will only
   // launch an instance if the instance service is available and it is enabled.