More graceful shutdown for ArcSession.

components/arc/arc_session.h

Index: components/arc/arc_session.h
diff --git a/components/arc/arc_session.h b/components/arc/arc_session.h
index 0e6e34b9dd23e702e8148310b8f061fc1c140327..37e23d6f03384486c842df303356ef55222289d9 100644
--- a/components/arc/arc_session.h
+++ b/components/arc/arc_session.h
@@ -8,9 +8,11 @@
 #include <memory>
 
 #include "base/macros.h"
+#include "base/memory/ref_counted.h"
 #include "base/observer_list.h"
 #include "base/sequenced_task_runner.h"
 #include "base/single_thread_task_runner.h"
+#include "base/task_runner.h"
 #include "components/arc/arc_bridge_service.h"
 
 namespace arc {
@@ -41,7 +43,8 @@ class ArcSession {
   };
 
   // Creates a default instance of ArcSession.
-  static std::unique_ptr<ArcSession> Create();
+  static std::unique_ptr<ArcSession> Create(
+      scoped_refptr<base::TaskRunner> blocking_task_runner);

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

can this also be const scoped_refptr<...>& if it does not interact badly with
the base::Closure?

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

Done.

   virtual ~ArcSession();
 
   // Starts and bootstraps a connection with the instance. The Observer's
@@ -53,6 +56,10 @@ class ArcSession {
   // The completion is notified via OnStopped() of the Delegate.
   virtual void Stop() = 0;
 
+  // Called when Chrome is in shutdown state. This is called when the message
+  // loop is already stopped, and the instance will soon be deleted.
+  virtual void OnShutdown() = 0;
+
   void AddObserver(Observer* observer);
   void RemoveObserver(Observer* observer);