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

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

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.h
diff --git a/chrome/browser/chromeos/arc/arc_auth_service.h b/chrome/browser/chromeos/arc/arc_auth_service.h
index c826427d37f2dfcaa63cec44649612867b937e31..957dd6f5f32fe53ad557fc818a57587c52a98657 100644
--- a/chrome/browser/chromeos/arc/arc_auth_service.h
+++ b/chrome/browser/chromeos/arc/arc_auth_service.h
@@ -40,13 +40,15 @@ class ArcAuthContext;
// This class proxies the request from the client to fetch an auth code from
// LSO.
-class ArcAuthService : public ArcService,
- public mojom::AuthHost,
- public ArcBridgeService::Observer,
- public ArcAndroidManagementCheckerDelegate,
- public ArcAuthContextDelegate,
- public syncable_prefs::PrefServiceSyncableObserver,
- public syncable_prefs::SyncedPrefObserver {
+class ArcAuthService
+ : public ArcService,
+ public mojom::AuthHost,
+ public ArcBridgeService::Observer,
+ public ArcBridgeService::InstanceObserver<mojom::AuthInstance>,
+ public ArcAndroidManagementCheckerDelegate,
+ public ArcAuthContextDelegate,
+ public syncable_prefs::PrefServiceSyncableObserver,
+ public syncable_prefs::SyncedPrefObserver {
public:
enum class State {
NOT_INITIALIZED, // Service is not initialized.
@@ -121,9 +123,12 @@ class ArcAuthService : public ArcService,
void RemoveObserver(Observer* observer);
// ArcBridgeService::Observer:
- void OnAuthInstanceReady() override;
void OnBridgeStopped() override;
+ // ArcBridgeService::InstanceObserver<mojom::AuthInstance>:
+ void OnInstanceReady(mojom::AuthInstance* auth_instance,
+ uint32_t version) override;
+
// AuthHost:
// For security reason this code can be used only once and exists for specific
// period of time.
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/arc_auth_service.cc » ('j') | chrome/browser/memory/tab_manager_delegate_chromeos.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698