arc-bridge: Introduce the ArcService class

chrome/browser/chromeos/arc/arc_auth_service.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "chrome/browser/chromeos/arc/arc_auth_service_impl.h"
+#include "chrome/browser/chromeos/arc/arc_auth_service.h"
 
 #include <utility>
 
 namespace arc {
 
-ArcAuthServiceImpl::ArcAuthServiceImpl() : binding_(this) {}
+ArcAuthService::ArcAuthService(ArcBridgeService* arc_bridge) : binding_(this) {
+  arc_bridge->AddObserver(this);
 
-ArcAuthServiceImpl::~ArcAuthServiceImpl() {
+  // If AuthInstance was ready before we AddObserver(), we won't get
+  // OnAuthInstanceReady events. For such case, we have to call it explicitly.
+  if (arc_bridge->auth_instance())
+    OnAuthInstanceReady();
+}
+
+ArcAuthService::~ArcAuthService() {
   ArcBridgeService::Get()->RemoveObserver(this);

[hidehiko, 2016/01/18 01:26:44]

For consistency, could you keep the |arc_bridge| passed to the constructor, and
call its RemoveObserver()?
Ditto for all other classes.

[Luis Héctor Chávez, 2016/01/19 17:43:54]

Done.

 }
 
-void ArcAuthServiceImpl::StartObservingBridgeServiceChanges() {
-  ArcBridgeService::Get()->AddObserver(this);
-}
-
-void ArcAuthServiceImpl::OnAuthInstanceReady() {
+void ArcAuthService::OnAuthInstanceReady() {
   arc::AuthHostPtr host;
   binding_.Bind(GetProxy(&host));
   ArcBridgeService::Get()->auth_instance()->Init(std::move(host));
 }
 
-void ArcAuthServiceImpl::GetAuthCode(const GetAuthCodeCallback& callback) {
+void ArcAuthService::GetAuthCode(const GetAuthCodeCallback& callback) {
   // TODO(victorhsieh): request auth code from LSO (crbug.com/571146).
   callback.Run(mojo::String("fake auth code from ArcAuthService in Chrome"));
 }
 
 }  // namespace arc