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

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

Issue 2490093002: Migrate opt-in auth flow to re-auth flow. (Closed)
Patch Set: rebase 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
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/arc_auth_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 891c278c25ecf953ff3a348e877c153ae6fcbf04..a8c07c5889fbcc2abba765367c9a0f6f3aaa4bcf 100644
--- a/chrome/browser/chromeos/arc/arc_auth_service.h
+++ b/chrome/browser/chromeos/arc/arc_auth_service.h
@@ -105,7 +105,6 @@ class ArcAuthService : public ArcService,
STOPPED,
SHOWING_TERMS_OF_SERVICE,
CHECKING_ANDROID_MANAGEMENT,
- FETCHING_CODE,
ACTIVE,
};
@@ -153,8 +152,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);
@@ -187,7 +184,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();
@@ -251,11 +248,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();
@@ -300,7 +298,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') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698