Add ArcAuthService to handle IPC request from ARC

components/arc/arc_service_manager.cc

Index: components/arc/arc_service_manager.cc
diff --git a/components/arc/arc_service_manager.cc b/components/arc/arc_service_manager.cc
index 0e18308aa106ba1c5455188d3aae6b61bb4c44fa..543ba5acce0427b11b7ae66bc83bd5b8dffaec73 100644
--- a/components/arc/arc_service_manager.cc
+++ b/components/arc/arc_service_manager.cc
@@ -8,6 +8,7 @@
 #include "base/thread_task_runner_handle.h"
 #include "components/arc/arc_bridge_bootstrap.h"
 #include "components/arc/arc_bridge_service_impl.h"
+#include "components/arc/auth/arc_auth_service.h"
 #include "components/arc/input/arc_input_bridge.h"
 #include "components/arc/power/arc_power_bridge.h"
 #include "components/arc/settings/arc_settings_bridge.h"
@@ -22,9 +23,11 @@ ArcServiceManager* g_arc_service_manager = nullptr;
 }  // namespace
 
 ArcServiceManager::ArcServiceManager(
+    scoped_ptr<ArcAuthService> auth_service,
     scoped_ptr<ArcSettingsBridge> settings_bridge)
     : arc_bridge_service_(
           new ArcBridgeServiceImpl(ArcBridgeBootstrap::Create())),
+      arc_auth_service_(std::move(auth_service)),

[dcheng, 2016/01/06 23:39:13]

Why not just instantiate it directly in here? Then you can just make the
constructor take a ArcBridgeService and AddObserver() in the ctor instead of
indirecting through the virtual method.

[victorhsieh, 2016/01/06 23:52:33]

It's just that we will need Profile to do the actual work (token exchange)
later.

[dcheng, 2016/01/07 00:02:33]

I think it would be simpler for ArcAuthService to just inherit
ArcBridgeService::Observer then. The indirection feels funky.

[victorhsieh, 2016/01/07 00:05:53]

Yeah, but that would make components depends on chrome, which is not ok :(

       arc_settings_bridge_(std::move(settings_bridge)) {
   DCHECK(!g_arc_service_manager);
   arc_input_bridge_ = ArcInputBridge::Create(arc_bridge_service_.get());
@@ -32,6 +35,7 @@ ArcServiceManager::ArcServiceManager(
   g_arc_service_manager = this;
 
   arc_settings_bridge_->StartObservingBridgeServiceChanges();
+  arc_auth_service_->StartObservingBridgeServiceChanges();
 }
 
 ArcServiceManager::~ArcServiceManager() {