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

Unified Diff: chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc

Issue 2677563005: Chromad: Use DM server reply to determine enrollment type (Closed)
Patch Set: comment+rebase+fix test Created 3 years, 10 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/ui/webui/chromeos/login/enrollment_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc
index d78961c37569dc7d716f2156d2f3be17c9c86767..17e6e0ea01c9d80320878a95fc8eb488b86bb347 100644
--- a/chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc
@@ -579,6 +579,7 @@ void EnrollmentScreenHandler::HandleAdDomainJoin(
authpolicy::ErrorType code) {
switch (code) {
case authpolicy::ERROR_NONE:
+ ShowEnrollmentSpinnerScreen();
controller_->OnAdJoined(gaia::ExtractDomainName(user_name));
Thiemo Nagel 2017/02/14 14:45:01 The email splitting in Chrome is different from em
Roman Sorokin (ftl) 2017/02/14 16:01:33 You mean output param from JoinAdDomain? Make sens
Thiemo Nagel 2017/02/14 16:43:05 I think it could be either an input or an output p
return;
case authpolicy::ERROR_UNKNOWN:

Powered by Google App Engine
This is Rietveld 408576698