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

Unified Diff: chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h

Issue 2898003002: Allow the powerwash shortcut only for devices without FRE (Closed)
Patch Set: Fixed review comments 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/login/enrollment/auto_enrollment_controller.h
diff --git a/chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h b/chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h
index b16deffebf0ebd082c80b1cc418cf4e4d547d59e..e903279e54f14a0b287cecb9711b3e389c063473 100644
--- a/chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h
+++ b/chrome/browser/chromeos/login/enrollment/auto_enrollment_controller.h
@@ -61,6 +61,18 @@ class AutoEnrollmentController {
// build status.
static Mode GetMode();
+ // Returns whether the auto-enrollment check is required. When
+ // kCheckEnrollmentKey VPD entry is present, it is explicitly stating whether
+ // the forced re-enrollment is required or not. Otherwise, for backward
+ // compatibility with devices upgrading from an older version of Chrome OS,
+ // the kActivateDateKey VPD entry is queried. If it's missing, FRE is not
+ // required. This enables factories to start full guest sessions for testing,
+ // see http://crbug.com/397354 for more context. The requirement for the
+ // machine serial number to be present is a sanity-check to ensure that the
+ // VPD has actually been read successfully. If VPD read failed, the FRE check
+ // is required.
+ static FRERequirement GetFRERequirement();
achuithb 2017/05/23 19:59:01 Why not have: bool IsForcedAutoEnrollmentRequired(
+
AutoEnrollmentController();
~AutoEnrollmentController();

Powered by Google App Engine
This is Rietveld 408576698