Refactor ArcBridgeServiceImpl part 2.

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 2607e641efd9e1cdfaa8cf2c0b966c50422b31ad..5e45d8a0f2092fc2202a3bfff4175a6a1b173d73 100644
--- a/components/arc/arc_bridge_service.h
+++ b/components/arc/arc_bridge_service.h
@@ -9,10 +9,8 @@
 #include <string>
 #include <vector>
 
-#include "base/files/scoped_file.h"
 #include "base/macros.h"
 #include "base/observer_list.h"
-#include "base/values.h"
 #include "components/arc/arc_session_observer.h"
 #include "components/arc/instance_holder.h"
 
@@ -131,7 +129,7 @@ class ArcBridgeService {
   InstanceHolder<mojom::WallpaperInstance>* wallpaper() { return &wallpaper_; }
 
   // Gets if ARC is currently running.
-  bool ready() const { return state() == State::READY; }
+  bool ready() const { return state() == State::RUNNING; }
 
   // Gets if ARC is currently stopped. This is not exactly !ready() since there
   // are transient states between ready() and stopped().
@@ -142,34 +140,32 @@ class ArcBridgeService {
   // in the following sequence:
   //
   // STOPPED
-  //   PrerequisitesChanged() ->
-  // CONNECTING
-  //   OnConnectionEstablished() ->
+  //   RequestStart() ->
+  // STARTING
+  //   OnSessionReady() ->
   // READY
   //
   // The ArcSession state machine can be thought of being substates of
-  // ArcBridgeService's CONNECTING state.
+  // ArcBridgeService's STARTING state.
+  // ArcBridgeService's state machien can be stopped at any phase.

[Luis Héctor Chávez, 2016/12/16 23:20:12]

nit: s/machien/machine/

[hidehiko, 2016/12/19 08:05:11]

Done.

   //
   // *
-  //   StopInstance() ->
+  //   RequestStop() ->
   // STOPPING
-  //   OnStopped() ->
+  //   OnSessionStopped() ->
   // STOPPED
   enum class State {
-    // ARC is not currently running.
+    // ARC instance is not currently running.
     STOPPED,
 
-    // The request to connect has been sent.
-    CONNECTING,
+    // Request to start ARC instance is received. Startin an ARC instance.

[Luis Héctor Chávez, 2016/12/16 23:20:12]

nit: s/Startin/Starting/

[hidehiko, 2016/12/19 08:05:11]

Done.

+    STARTING,
 
-    // The instance has started, and the bridge is fully established.
-    CONNECTED,
+    // ARC instance has finished initializing, and is now ready for interaction
+    // with other services.
+    RUNNING,
 
-    // The ARC instance has finished initializing and is now ready for user
-    // interaction.
-    READY,
-
-    // The ARC instance has started shutting down.
+    // Request to stop ARC instance is recieved. Stopping the ARC instance.
     STOPPING,
   };