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

Unified Diff: chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.h

Issue 2844383006: Turn ArcSupportHost from Observer model to Delegate (Closed)
Patch Set: rebase Created 3 years, 6 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/optin/arc_terms_of_service_default_negotiator.h
diff --git a/chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.h b/chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.h
index 10d8a0dc0b8b1690fda14f686573f6854ed03f52..7e2e95bd02cf10af468a7e5889654ba92ce0e4d7 100644
--- a/chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.h
+++ b/chrome/browser/chromeos/arc/optin/arc_terms_of_service_default_negotiator.h
@@ -23,7 +23,7 @@ class ArcOptInPreferenceHandler;
// Handles the Terms-of-service agreement user action via default OptIn UI.
class ArcTermsOfServiceDefaultNegotiator
: public ArcTermsOfServiceNegotiator,
- public ArcSupportHost::Observer,
+ public ArcSupportHost::TermsOfServiceDelegate,
public ArcOptInPreferenceHandlerObserver {
public:
ArcTermsOfServiceDefaultNegotiator(PrefService* pref_service,
@@ -31,14 +31,12 @@ class ArcTermsOfServiceDefaultNegotiator
~ArcTermsOfServiceDefaultNegotiator() override;
private:
- // ArcSupportHost::Observer:
- void OnWindowClosed() override;
+ // ArcSupportHost::TermsOfServiceDelegate:
void OnTermsAgreed(bool is_metrics_enabled,
bool is_backup_and_restore_enabled,
bool is_location_service_enabled) override;
- void OnAuthSucceeded(const std::string& auth_code) override;
- void OnRetryClicked() override;
- void OnSendFeedbackClicked() override;
+ void OnTermsRejected() override;
+ void OnTermsRetryClicked() override;
// ArcOptInPreferenceHandlerObserver:
void OnMetricsModeChanged(bool enabled, bool managed) override;

Powered by Google App Engine
This is Rietveld 408576698