Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1862)

Unified Diff: chrome/browser/chromeos/arc/arc_auth_service.cc

Issue 2133503002: arc: Revamp the ArcBridgeService interface (Closed) Base URL: https://chromium.googlesource.com/a/chromium/src.git@master
Patch Set: Fix ui_arc_unittests Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/arc/arc_auth_service.cc
diff --git a/chrome/browser/chromeos/arc/arc_auth_service.cc b/chrome/browser/chromeos/arc/arc_auth_service.cc
index 6ac8869e8447b5acb705e45fd1aa9b919e218f44..57780eb288f9705bb1ccb7ff93bc8c54558781ae 100644
--- a/chrome/browser/chromeos/arc/arc_auth_service.cc
+++ b/chrome/browser/chromeos/arc/arc_auth_service.cc
@@ -101,6 +101,7 @@ ArcAuthService::ArcAuthService(ArcBridgeService* bridge_service)
arc_auth_service = this;
arc_bridge_service()->AddObserver(this);
+ arc_bridge_service()->auth()->AddObserver(this);
}
ArcAuthService::~ArcAuthService() {
@@ -108,6 +109,7 @@ ArcAuthService::~ArcAuthService() {
DCHECK(arc_auth_service == this);
Shutdown();
+ arc_bridge_service()->auth()->RemoveObserver(this);
arc_bridge_service()->RemoveObserver(this);
arc_auth_service = nullptr;
@@ -190,9 +192,9 @@ bool ArcAuthService::IsAllowedForProfile(const Profile* profile) {
return true;
}
-void ArcAuthService::OnAuthInstanceReady() {
- arc_bridge_service()->auth_instance()->Init(
- binding_.CreateInterfacePtrAndBind());
+void ArcAuthService::OnInstanceReady(mojom::AuthInstance* auth_instance,
+ uint32_t version) {
+ auth_instance->Init(binding_.CreateInterfacePtrAndBind());
}
void ArcAuthService::OnBridgeStopped() {

Powered by Google App Engine
This is Rietveld 408576698