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

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

Issue 2844383006: Turn ArcSupportHost from Observer model to Delegate (Closed)
Patch Set: Turn ArcSupportHost from Observer model to Delegate Created 3 years, 7 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_session_manager.h
diff --git a/chrome/browser/chromeos/arc/arc_session_manager.h b/chrome/browser/chromeos/arc/arc_session_manager.h
index 586dbc6ee5b1e2b082652c0950d0c538fb024de8..e7c747ca912e9a55973d8210e2ea28534d423e0c 100644
--- a/chrome/browser/chromeos/arc/arc_session_manager.h
+++ b/chrome/browser/chromeos/arc/arc_session_manager.h
@@ -35,7 +35,7 @@ enum class ProvisioningResult : int;
// This class proxies the request from the client to fetch an auth code from
// LSO. It lives on the UI thread.
class ArcSessionManager : public ArcSessionRunner::Observer,
- public ArcSupportHost::Observer {
+ public ArcSupportHost::ErrorDelegate {
public:
// Represents each State of ARC session.
// NOT_INITIALIZED: represents the state that the Profile is not yet ready
@@ -194,11 +194,8 @@ class ArcSessionManager : public ArcSessionRunner::Observer,
// periodically.
void RecordArcState();
- // ArcSupportHost::Observer:
- void OnWindowClosed() override;
- void OnTermsAgreed(bool is_metrics_enabled,
- bool is_backup_and_restore_enabled,
- bool is_location_service_enabled) override;
+ // ArcSupportHost:::ErrorDelegate:
+ void OnOptInAborted() override;
void OnRetryClicked() override;
void OnSendFeedbackClicked() override;

Powered by Google App Engine
This is Rietveld 408576698