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

Side by Side Diff: chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.cc

Issue 1412813003: This CL replaces user_manager::UserID with AccountId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@468875--Chrome-OS-handles-deletion-of-Gmail-account-poorly--Create-AccountID-structure-part2--user_names
Patch Set: Fix Win GN 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/chromeos/policy/user_cloud_policy_manager_chromeos.h" 5 #include "chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.h"
6 6
7 #include <set> 7 #include <set>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 void OnWildcardCheckCompleted(const std::string& username, 62 void OnWildcardCheckCompleted(const std::string& username,
63 WildcardLoginChecker::Result result) { 63 WildcardLoginChecker::Result result) {
64 if (result == WildcardLoginChecker::RESULT_BLOCKED) { 64 if (result == WildcardLoginChecker::RESULT_BLOCKED) {
65 LOG(ERROR) << "Online wildcard login check failed, terminating session."; 65 LOG(ERROR) << "Online wildcard login check failed, terminating session.";
66 66
67 // TODO(mnissler): This only removes the user pod from the login screen, but 67 // TODO(mnissler): This only removes the user pod from the login screen, but
68 // the cryptohome remains. This is because deleting the cryptohome for a 68 // the cryptohome remains. This is because deleting the cryptohome for a
69 // logged-in session is not possible. Fix this either by delaying the 69 // logged-in session is not possible. Fix this either by delaying the
70 // cryptohome deletion operation or by getting rid of the in-session 70 // cryptohome deletion operation or by getting rid of the in-session
71 // wildcard check. 71 // wildcard check.
72 user_manager::UserManager::Get()->RemoveUserFromList(username); 72 user_manager::UserManager::Get()->RemoveUserFromList(
73 AccountId::FromUserEmail(username));
73 chrome::AttemptUserExit(); 74 chrome::AttemptUserExit();
74 } 75 }
75 } 76 }
76 77
77 } // namespace 78 } // namespace
78 79
79 UserCloudPolicyManagerChromeOS::UserCloudPolicyManagerChromeOS( 80 UserCloudPolicyManagerChromeOS::UserCloudPolicyManagerChromeOS(
80 scoped_ptr<CloudPolicyStore> store, 81 scoped_ptr<CloudPolicyStore> store,
81 scoped_ptr<CloudExternalDataManager> external_data_manager, 82 scoped_ptr<CloudExternalDataManager> external_data_manager,
82 const base::FilePath& component_policy_cache_path, 83 const base::FilePath& component_policy_cache_path,
(...skipping 333 matching lines...) Expand 10 before | Expand all | Expand 10 after
416 // OnComponentCloudPolicyUpdated() once it's ready. 417 // OnComponentCloudPolicyUpdated() once it's ready.
417 return; 418 return;
418 } 419 }
419 420
420 core()->StartRefreshScheduler(); 421 core()->StartRefreshScheduler();
421 core()->TrackRefreshDelayPref(local_state_, 422 core()->TrackRefreshDelayPref(local_state_,
422 policy_prefs::kUserPolicyRefreshRate); 423 policy_prefs::kUserPolicyRefreshRate);
423 } 424 }
424 425
425 } // namespace policy 426 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698