Do not try to enroll past OOBE/enrollment.

Do not try to enroll past OOBE/enrollment.

BUG=647480
TEST=manual

Committed: https://crrev.com/ebafc22b74ceef8bb436396bf3194b8759c48a59
Cr-Commit-Position: refs/heads/master@{#419302}

[The one and only Dr. Crash, 2016-09-16 00:03:55]

drcrash@chromium.org changed reviewers:
+ xiyuan@chromium.org

[The one and only Dr. Crash, 2016-09-16 06:32:33]

On 2016/09/16 00:03:55, The one and only Dr. Crash wrote:

This ensures that EnrollmentConfig::should_enroll() won't return true due to
attestation enrollment after the OOBE has run. This is based on the premise that
at this time (where ZTE is either enabled as an optional feature or forced), if
one goes to the end of OOBE and ZTE was optional, then ZTE shouldn't be tried
again because the user explicitly opted out of it during the first OOBE.

[xiyuan, 2016-09-16 07:49:13]

lgtm

https://codereview.chromium.org/2340933004/diff/1/chrome/browser/chromeos/pol...
File chrome/browser/chromeos/policy/device_cloud_policy_initializer.cc (right):

https://codereview.chromium.org/2340933004/diff/1/chrome/browser/chromeos/pol...
chrome/browser/chromeos/policy/device_cloud_policy_initializer.cc:157:
config.auth_mechanism = EnrollmentConfig::AUTH_MECHANISM_INTERACTIVE;
nit: wrap with {} since the condition takes more than one line.

[The one and only Dr. Crash, 2016-09-16 17:27:41]

The CQ bit was checked by drcrash@chromium.org

[commit-bot: I haz the power, 2016-09-16 17:28:20]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 18:06:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-16 18:06:37]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[The one and only Dr. Crash, 2016-09-16 21:07:28]

The CQ bit was checked by drcrash@chromium.org

[The one and only Dr. Crash, 2016-09-16 21:07:29]

The patchset sent to the CQ was uploaded after l-g-t-m from xiyuan@chromium.org
Link to the patchset: https://codereview.chromium.org/2340933004/#ps20001
(title: "Forgot unit tests. (Oops.)")

[commit-bot: I haz the power, 2016-09-16 21:07:55]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 22:29:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-16 22:32:31]

Description was changed from

==========
Do not try to enroll past OOBE/enrollment.

BUG=647480
TEST=manual
==========

to

==========
Do not try to enroll past OOBE/enrollment.

BUG=647480
TEST=manual

Committed: https://crrev.com/ebafc22b74ceef8bb436396bf3194b8759c48a59
Cr-Commit-Position: refs/heads/master@{#419302}
==========

[commit-bot: I haz the power, 2016-09-16 22:32:32]

Patchset 2 (id:??) landed as
https://crrev.com/ebafc22b74ceef8bb436396bf3194b8759c48a59
Cr-Commit-Position: refs/heads/master@{#419302}