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

Unified Diff: chrome/browser/chromeos/settings/session_manager_operation.h

Issue 2801993002: Abandon user sign in when policy is retrieved before session started (Closed)
Patch Set: Nits Created 3 years, 8 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/settings/session_manager_operation.h
diff --git a/chrome/browser/chromeos/settings/session_manager_operation.h b/chrome/browser/chromeos/settings/session_manager_operation.h
index 3b67b31d107b70fba69c87f2c27e4d8af379c218..425706eb6375f7a00818d3b95b76b70ef34788ea 100644
--- a/chrome/browser/chromeos/settings/session_manager_operation.h
+++ b/chrome/browser/chromeos/settings/session_manager_operation.h
@@ -12,6 +12,7 @@
#include "base/memory/ref_counted.h"
#include "chrome/browser/chromeos/policy/device_cloud_policy_validator.h"
#include "chrome/browser/chromeos/settings/device_settings_service.h"
+#include "chromeos/dbus/session_manager_client.h"
#include "net/cert/x509_util_nss.h"
namespace enterprise_management {
@@ -27,8 +28,6 @@ class PublicKey;
namespace chromeos {
-class SessionManagerClient;
-
// Handles a single transaction with session manager. This is a virtual base
// class that contains common infrastructure for key and policy loading. There
// are subclasses for loading, storing and signing policy blobs.
@@ -111,7 +110,9 @@ class SessionManagerOperation {
void BlockingRetrieveDeviceSettings();
// Validates device settings after retrieval from session_manager.
- void ValidateDeviceSettings(const std::string& policy_blob);
+ void ValidateDeviceSettings(
+ const std::string& policy_blob,
+ chromeos::SessionManagerClient::RetrievePolicyResponseType response_type);
Daniel Erat 2017/04/24 19:50:13 you don't need chromeos:: here; this is already in
igorcov 2017/04/25 09:18:42 Done.
// Extracts status and device settings from the validator and reports them.
void ReportValidatorStatus(policy::DeviceCloudPolicyValidator* validator);

Powered by Google App Engine
This is Rietveld 408576698