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

Unified Diff: chrome/browser/chromeos/policy/user_cloud_policy_store_chromeos.cc

Issue 2890433002: Revert "Abandon user sign in when policy is retrieved before session started." (Closed)
Patch Set: Created 3 years, 7 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/policy/user_cloud_policy_store_chromeos.cc
diff --git a/chrome/browser/chromeos/policy/user_cloud_policy_store_chromeos.cc b/chrome/browser/chromeos/policy/user_cloud_policy_store_chromeos.cc
index f3fd2ae7afda63bb8387527039bbfb1972510544..f73ec0ceb7924b4cee34ba619a98db1525294b92 100644
--- a/chrome/browser/chromeos/policy/user_cloud_policy_store_chromeos.cc
+++ b/chrome/browser/chromeos/policy/user_cloud_policy_store_chromeos.cc
@@ -19,17 +19,14 @@
#include "base/stl_util.h"
#include "base/strings/stringprintf.h"
#include "chrome/browser/chromeos/policy/user_policy_token_loader.h"
-#include "chrome/browser/lifetime/application_lifetime.h"
#include "chromeos/cryptohome/cryptohome_parameters.h"
#include "chromeos/dbus/cryptohome_client.h"
+#include "chromeos/dbus/session_manager_client.h"
#include "components/policy/core/common/cloud/cloud_policy_constants.h"
#include "components/policy/proto/cloud_policy.pb.h"
#include "components/policy/proto/device_management_local.pb.h"
#include "google_apis/gaia/gaia_auth_util.h"
-using RetrievePolicyResponseType =
- chromeos::SessionManagerClient::RetrievePolicyResponseType;
-
namespace em = enterprise_management;
namespace policy {
@@ -112,18 +109,9 @@ void UserCloudPolicyStoreChromeOS::LoadImmediately() {
// However, on those paths we must load policy synchronously so that the
// Profile initialization never sees unmanaged prefs, which would lead to
// data loss. http://crbug.com/263061
- std::string policy_blob;
- RetrievePolicyResponseType response_type =
+ std::string policy_blob =
session_manager_client_->BlockingRetrievePolicyForUser(
- cryptohome::Identification(account_id_), &policy_blob);
-
- if (response_type == RetrievePolicyResponseType::SESSION_DOES_NOT_EXIST) {
- LOG(ERROR)
- << "Session manager claims that session doesn't exist; signing out";
- chrome::AttemptUserExit();
- return;
- }
-
+ cryptohome::Identification(account_id_));
if (policy_blob.empty()) {
// The session manager doesn't have policy, or the call failed.
NotifyStoreLoaded();
@@ -226,19 +214,7 @@ void UserCloudPolicyStoreChromeOS::OnPolicyStored(bool success) {
}
void UserCloudPolicyStoreChromeOS::OnPolicyRetrieved(
- const std::string& policy_blob,
- RetrievePolicyResponseType response_type) {
- // Disallow the sign in when the Chrome OS user session has not started, which
- // should always happen before the profile construction. An attempt to read
- // the policy outside the session will always fail and return an empty policy
- // blob.
- if (response_type == RetrievePolicyResponseType::SESSION_DOES_NOT_EXIST) {
- LOG(ERROR)
- << "Session manager claims that session doesn't exist; signing out";
- chrome::AttemptUserExit();
- return;
- }
-
+ const std::string& policy_blob) {
if (policy_blob.empty()) {
// session_manager doesn't have policy. Adjust internal state and notify
// the world about the policy update.

Powered by Google App Engine
This is Rietveld 408576698