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

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

Issue 2490093002: Migrate opt-in auth flow to re-auth flow. (Closed)
Patch Set: Fix retry. Created 4 years, 1 month 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 a94c688339df30131523cedaad2f733abb719f00..040061f7a3a702b4272ebe0109ffff7f182255ab 100644
--- a/chrome/browser/chromeos/arc/arc_auth_service.h
+++ b/chrome/browser/chromeos/arc/arc_auth_service.h
@@ -101,7 +101,6 @@ class ArcAuthService : public ArcService,
STOPPED,
SHOWING_TERMS_OF_SERVICE,
CHECKING_ANDROID_MANAGEMENT,
- FETCHING_CODE,
ACTIVE,
};
@@ -149,8 +148,6 @@ class ArcAuthService : public ArcService,
State state() const { return state_; }
- std::string GetAndResetAuthCode();
-
// Adds or removes observers.
void AddObserver(Observer* observer);
void RemoveObserver(Observer* observer);
@@ -183,7 +180,7 @@ class ArcAuthService : public ArcService,
void OnSignInFailedInternal(ProvisioningResult result);
// Called from Arc support platform app to set auth code and start arc.
- void SetAuthCodeAndStartArc(const std::string& auth_code);
+ void OnAuthCodeObtained(const std::string& auth_code);
// Called from Arc support platform app when user cancels signing.
void CancelAuthCode();
@@ -247,11 +244,12 @@ class ArcAuthService : public ArcService,
ArcSupportHost* support_host() { return support_host_.get(); }
+ void StartArc();
+
private:
using AccountInfoCallback = base::Callback<void(mojom::AccountInfoPtr)>;
class AccountInfoNotifier;
- void StartArc();
// TODO(hidehiko): move UI methods/fields to ArcSupportHost.
void ShowUI(ArcSupportHost::UIPage page, const base::string16& status);
void CloseUI();
@@ -296,7 +294,6 @@ class ArcAuthService : public ArcService,
State state_ = State::NOT_INITIALIZED;
base::ObserverList<Observer> observer_list_;
std::unique_ptr<ArcAppLauncher> playstore_launcher_;
- std::string auth_code_;
ArcSupportHost::UIPage ui_page_ = ArcSupportHost::UIPage::NO_PAGE;
base::string16 ui_page_status_;
bool clear_required_ = false;
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/arc_auth_service.cc » ('j') | chrome/browser/chromeos/arc/arc_auth_service.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698