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

Side by Side Diff: chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc

Issue 1440583002: This CL replaces e-mail with AccountId on user selection screen. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix build. Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.h" 5 #include "chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.h"
6 6
7 #include "ash/system/chromeos/devicetype_utils.h" 7 #include "ash/system/chromeos/devicetype_utils.h"
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/guid.h" 9 #include "base/guid.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 816 matching lines...) Expand 10 before | Expand all | Expand 10 after
827 VLOG(1) << "LoadAuthExtension, force: " << force 827 VLOG(1) << "LoadAuthExtension, force: " << force
828 << ", offline: " << offline; 828 << ", offline: " << offline;
829 GaiaContext context; 829 GaiaContext context;
830 context.force_reload = force; 830 context.force_reload = force;
831 context.is_local = offline; 831 context.is_local = offline;
832 context.use_offline = offline; 832 context.use_offline = offline;
833 context.email = populated_email_; 833 context.email = populated_email_;
834 context.is_enrolling_consumer_management = is_enrolling_consumer_management_; 834 context.is_enrolling_consumer_management = is_enrolling_consumer_management_;
835 835
836 std::string gaia_id; 836 std::string gaia_id;
837 if (user_manager::UserManager::Get()->FindGaiaID( 837 if (!context.email.empty() &&
838 AccountId::FromUserEmail(context.email), &gaia_id)) 838 user_manager::UserManager::Get()->FindGaiaID(
839 AccountId::FromUserEmail(context.email), &gaia_id)) {
839 context.gaia_id = gaia_id; 840 context.gaia_id = gaia_id;
841 }
840 842
841 if (Delegate()) { 843 if (Delegate()) {
842 context.show_users = Delegate()->IsShowUsers(); 844 context.show_users = Delegate()->IsShowUsers();
843 } 845 }
844 846
845 if (!context.email.empty()) { 847 if (!context.email.empty()) {
846 context.gaps_cookie = 848 context.gaps_cookie =
847 user_manager::UserManager::Get()->GetKnownUserGAPSCookie( 849 user_manager::UserManager::Get()->GetKnownUserGAPSCookie(
848 AccountId::FromUserEmail(gaia::CanonicalizeEmail(context.email))); 850 AccountId::FromUserEmail(gaia::CanonicalizeEmail(context.email)));
849 } 851 }
(...skipping 15 matching lines...) Expand all
865 bool GaiaScreenHandler::IsRestrictiveProxy() const { 867 bool GaiaScreenHandler::IsRestrictiveProxy() const {
866 return !disable_restrictive_proxy_check_for_test_ && 868 return !disable_restrictive_proxy_check_for_test_ &&
867 !IsOnline(captive_portal_status_); 869 !IsOnline(captive_portal_status_);
868 } 870 }
869 871
870 void GaiaScreenHandler::DisableRestrictiveProxyCheckForTest() { 872 void GaiaScreenHandler::DisableRestrictiveProxyCheckForTest() {
871 disable_restrictive_proxy_check_for_test_ = true; 873 disable_restrictive_proxy_check_for_test_ = true;
872 } 874 }
873 875
874 } // namespace chromeos 876 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698