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

Unified Diff: chrome/browser/chromeos/login/screens/controller_pairing_screen.cc

Issue 547503002: Redirect to the enterprise enrollment screen during remora and shark pairing (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Code review fix. Created 6 years, 3 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/login/screens/controller_pairing_screen.cc
diff --git a/chrome/browser/chromeos/login/screens/controller_pairing_screen.cc b/chrome/browser/chromeos/login/screens/controller_pairing_screen.cc
index 382ad45be58bd495837f7866c32e744b537f24dd..6c242ab8d930bda274a8dca7246c59aa14d21124 100644
--- a/chrome/browser/chromeos/login/screens/controller_pairing_screen.cc
+++ b/chrome/browser/chromeos/login/screens/controller_pairing_screen.cc
@@ -12,10 +12,6 @@
using namespace chromeos::controller_pairing;
using namespace pairing_chromeos;
-namespace {
-const char* kTestAuthToken = "TestAuthToken";
-};
-
namespace chromeos {
ControllerPairingScreen::ControllerPairingScreen(
@@ -112,6 +108,9 @@ void ControllerPairingScreen::PairingStageChanged(Stage new_stage) {
break;
}
case ControllerPairingController::STAGE_WAITING_FOR_CREDENTIALS: {
dzhioev (left Google) 2014/09/09 01:35:41 You need to remove |this| from |controller_|'s obs
Zachary Kuznia 2014/09/09 02:17:51 Done.
+ get_screen_observer()->OnExit(
+ WizardController::CONTROLLER_PAIRING_FINISHED);
+ // TODO: Move the rest of the stages to the proper location.
desired_page = kPageEnrollmentIntroduction;
break;
}
@@ -200,8 +199,6 @@ void ControllerPairingScreen::OnUserActed(const std::string& action) {
gaia::SanitizeEmail(context_.GetString(kContextKeyAccountId));
const std::string domain(gaia::ExtractDomainName(account_id));
context_.SetString(kContextKeyEnrollmentDomain, domain);
- // TODO(zork): Get proper credentials. (http://crbug.com/405744)
- controller_->OnAuthenticationDone(domain, kTestAuthToken);
} else if (action == kActionStartSession) {
controller_->StartSession();
}

Powered by Google App Engine
This is Rietveld 408576698