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

Unified Diff: chrome/browser/policy/cloud/cloud_policy_validator.h

Issue 24041002: Turn off future-timestamp cloud policy checks on desktop (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More fixes. Created 7 years, 3 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/policy/cloud/cloud_policy_validator.h
diff --git a/chrome/browser/policy/cloud/cloud_policy_validator.h b/chrome/browser/policy/cloud/cloud_policy_validator.h
index d53a0c6f91327664d10a157818a61085a7a542e0..08092d6361c9fa6cb923f0f2e13b94e827efb23f 100644
--- a/chrome/browser/policy/cloud/cloud_policy_validator.h
+++ b/chrome/browser/policy/cloud/cloud_policy_validator.h
@@ -79,9 +79,14 @@ class CloudPolicyValidatorBase {
};
enum ValidateTimestampOption {
- // The policy must have a timestamp field.
+ // The policy must have a timestamp field and it should be checked against
+ // both the start and end times.
TIMESTAMP_REQUIRED,
+ // The system time is unreliable so we should not validate against the
+ // system timestamp.
+ TIMESTAMP_REQUIRED_BUT_SYSTEM_TIME_UNRELIABLE,
Mattias Nissler (ping if slow) 2013/09/09 08:00:41 Naming here is a bit unfortunate given that this c
+
// No timestamp field is required.
TIMESTAMP_NOT_REQUIRED,
};

Powered by Google App Engine
This is Rietveld 408576698