Don't start FRE from the login screen

chrome/browser/chromeos/policy/auto_enrollment_client.h

Index: chrome/browser/chromeos/policy/auto_enrollment_client.h
diff --git a/chrome/browser/chromeos/policy/auto_enrollment_client.h b/chrome/browser/chromeos/policy/auto_enrollment_client.h
index 91e504da9d1315569c30e88011aa35a008611253..ec3d931c56443483ed146ebe19c7cdf97214eecd 100644
--- a/chrome/browser/chromeos/policy/auto_enrollment_client.h
+++ b/chrome/browser/chromeos/policy/auto_enrollment_client.h
@@ -34,20 +34,21 @@ namespace policy {
 class DeviceManagementRequestJob;
 class DeviceManagementService;
 
-// Indicates the current state of the auto-enrollment check.
+// Indicates the current state of the auto-enrollment check.  (Numeric values
+// are just to make reading of log files easier.)
 enum AutoEnrollmentState {

[achuithb, 2017/04/19 19:04:43]

Maybe not in this CL, but we should consider making this an enum class, and
changing the enum values below to kIdle, kPending, kConnectionError,
kServerError, kTriggerEnrollment, kNoEnrollment.

For logging, it might be nicer to have a function like: 
const std::string& AutoEnrollmentStateString(AutoEnrollmentState state);

Again, not necessary for this CL.

[Thiemo Nagel, 2017/04/20 13:24:00]

I agree that what you suggest is a nicer approach, though why not go all the way
and overload the << operator for logging?

std::ostream& operator<<(std::ostream&, const AutoEnrollmentSate&).

Otoh, adding all the strings bloats the binary and I'm not entirely sure this is
justified.

   // Not yet started.
-  AUTO_ENROLLMENT_STATE_IDLE,
+  AUTO_ENROLLMENT_STATE_IDLE = 0,
   // Working, another event will be fired eventually.
-  AUTO_ENROLLMENT_STATE_PENDING,
+  AUTO_ENROLLMENT_STATE_PENDING = 1,
   // Failed to connect to DMServer.
-  AUTO_ENROLLMENT_STATE_CONNECTION_ERROR,
+  AUTO_ENROLLMENT_STATE_CONNECTION_ERROR = 2,
   // Connection successful, but the server failed to generate a valid reply.
-  AUTO_ENROLLMENT_STATE_SERVER_ERROR,
+  AUTO_ENROLLMENT_STATE_SERVER_ERROR = 3,
   // Check completed successfully, enrollment should be triggered.
-  AUTO_ENROLLMENT_STATE_TRIGGER_ENROLLMENT,
+  AUTO_ENROLLMENT_STATE_TRIGGER_ENROLLMENT = 4,
   // Check completed successfully, enrollment not applicable.
-  AUTO_ENROLLMENT_STATE_NO_ENROLLMENT,
+  AUTO_ENROLLMENT_STATE_NO_ENROLLMENT = 5,
 };
 
 // Interacts with the device management service and determines whether this