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

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

Issue 1425093004: Revert of 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: 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( 72 user_manager::UserManager::Get()->RemoveUserFromList(username);
73 AccountId::FromUserEmail(username));
74 chrome::AttemptUserExit(); 73 chrome::AttemptUserExit();
75 } 74 }
76 } 75 }
77 76
78 } // namespace 77 } // namespace
79 78
80 UserCloudPolicyManagerChromeOS::UserCloudPolicyManagerChromeOS( 79 UserCloudPolicyManagerChromeOS::UserCloudPolicyManagerChromeOS(
81 scoped_ptr<CloudPolicyStore> store, 80 scoped_ptr<CloudPolicyStore> store,
82 scoped_ptr<CloudExternalDataManager> external_data_manager, 81 scoped_ptr<CloudExternalDataManager> external_data_manager,
83 const base::FilePath& component_policy_cache_path, 82 const base::FilePath& component_policy_cache_path,
(...skipping 333 matching lines...) Expand 10 before | Expand all | Expand 10 after
417 // OnComponentCloudPolicyUpdated() once it's ready. 416 // OnComponentCloudPolicyUpdated() once it's ready.
418 return; 417 return;
419 } 418 }
420 419
421 core()->StartRefreshScheduler(); 420 core()->StartRefreshScheduler();
422 core()->TrackRefreshDelayPref(local_state_, 421 core()->TrackRefreshDelayPref(local_state_,
423 policy_prefs::kUserPolicyRefreshRate); 422 policy_prefs::kUserPolicyRefreshRate);
424 } 423 }
425 424
426 } // namespace policy 425 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698