Add UI components for license type selection in ChromeOS login UI.

Add UI components for license type selection in ChromeOS login UI.
Changes that would actually use them (enrollment screen) would come in next CL.

BUG=725124
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2959853003
Cr-Commit-Position: refs/heads/master@{#489529}
Committed: https://chromium.googlesource.com/chromium/src/+/3d1e27d07f9eb13b298556a877d4b54bcb94ca33

[Denis Kuznetsov (DE-MUC), 2017-06-27 16:18:13]

Description was changed from

==========
Add UI components for license type selection in ChromeOS login UI.
Changes that would actually use them (enrollment screen) would come in next CL.

BUG=725124
==========

to

==========
Add UI components for license type selection in ChromeOS login UI.
Changes that would actually use them (enrollment screen) would come in next CL.

BUG=725124
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Denis Kuznetsov (DE-MUC), 2017-06-27 16:21:15]

antrim@chromium.org changed reviewers:
+ alemate@chromium.org, bartfab@chromium.org, nkostylev@chromium.org

[Denis Kuznetsov (DE-MUC), 2017-06-27 16:21:16]

bartfab:
  chrome/browser/chromeos/policy/enrollment_status_chromeos.h
  components/policy/core/common/cloud/enterprise_metrics.h
alemate:
  chrome/browser/chromeos/login/enrollment/enrollment_screen.cc
  chrome/browser/chromeos/login/enrollment/enrollment_screen.h
  chrome/browser/chromeos/login/enrollment/enrollment_screen_view.h
  chrome/browser/chromeos/login/enrollment/enterprise_enrollment_helper_impl.cc
  chrome/browser/chromeos/login/enrollment/mock_enrollment_screen.h
  chrome/browser/resources/chromeos/login/custom_elements_login.html
  chrome/browser/resources/chromeos/login/custom_elements_login.js
  chrome/browser/resources/chromeos/login/custom_elements_oobe.html
  chrome/browser/resources/chromeos/login/custom_elements_oobe.js
  chrome/browser/resources/chromeos/login/enrollment_license_card.css
  chrome/browser/resources/chromeos/login/enrollment_license_card.html
  chrome/browser/resources/chromeos/login/enrollment_license_card.js
  chrome/browser/resources/chromeos/login/oobe_screen_oauth_enrollment.css
  chrome/browser/resources/chromeos/login/oobe_screen_oauth_enrollment.html
  chrome/browser/resources/chromeos/login/oobe_screen_oauth_enrollment.js
  chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.cc
  chrome/browser/ui/webui/chromeos/login/enrollment_screen_handler.h
nkostylev:
  chrome/browser/chromeos/login/enrollment/enrollment_screen.cc
  chrome/browser/chromeos/login/enrollment/enrollment_screen.h
  chrome/browser/chromeos/login/enrollment/enrollment_screen_view.h
  chrome/browser/chromeos/login/enrollment/enterprise_enrollment_helper_impl.cc
  chrome/browser/chromeos/login/enrollment/mock_enrollment_screen.h
  chrome/browser/chromeos/policy/enrollment_status_chromeos.h

[Denis Kuznetsov (DE-MUC), 2017-06-27 16:21:42]

The CQ bit was checked by antrim@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-27 16:21:57]

Dry run: CQ is trying da patch.

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

[Nikita (slow), 2017-06-27 17:35:03]

On 2017/06/27 16:21:16, Denis Kuznetsov (DE-MUC) wrote:
> nkostylev:
>   chrome/browser/chromeos/login/enrollment/enrollment_screen.cc
>   chrome/browser/chromeos/login/enrollment/enrollment_screen.h
>   chrome/browser/chromeos/login/enrollment/enrollment_screen_view.h
>  
chrome/browser/chromeos/login/enrollment/enterprise_enrollment_helper_impl.cc
>   chrome/browser/chromeos/login/enrollment/mock_enrollment_screen.h
>   chrome/browser/chromeos/policy/enrollment_status_chromeos.h

lgtm

[commit-bot: I haz the power, 2017-06-27 17:44:55]

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

[commit-bot: I haz the power, 2017-06-27 17:44:56]

Dry run: This issue passed the CQ dry run.

[Alexander Alekseev, 2017-07-06 09:36:01]

lgtm

[Denis Kuznetsov (DE-MUC), 2017-07-06 18:47:14]

antrim@chromium.org changed reviewers:
+ emaxx@chromium.org
- bartfab@chromium.org

[Denis Kuznetsov (DE-MUC), 2017-07-06 18:47:14]

+emaxx as a reviewer instead of bartfab

[emaxx, 2017-07-07 14:32:20]

policy - lgtm with a nit

[emaxx, 2017-07-07 14:32:34]

https://codereview.chromium.org/2959853003/diff/1/components/policy/core/comm...
File components/policy/core/common/cloud/enterprise_metrics.h (right):

https://codereview.chromium.org/2959853003/diff/1/components/policy/core/comm...
components/policy/core/common/cloud/enterprise_metrics.h:233:
kEnrollmentLicenseRequestFailed = 57,
nit: s/kEnrollmentLicenseRequestFailed/kMetricEnrollmentLicenseRequestFailed/
(following the style of the previous 56 items of the enum)

[Denis Kuznetsov (DE-MUC), 2017-07-13 18:43:43]

The CQ bit was checked by antrim@chromium.org

[Denis Kuznetsov (DE-MUC), 2017-07-13 18:43:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
alemate@chromium.org, nkostylev@chromium.org, emaxx@chromium.org
Link to the patchset: https://codereview.chromium.org/2959853003/#ps20001
(title: "Rebased to ToT")

[commit-bot: I haz the power, 2017-07-13 18:44:01]

CQ is trying da patch.

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

[Denis Kuznetsov (DE-MUC), 2017-07-13 18:44:03]

https://codereview.chromium.org/2959853003/diff/1/components/policy/core/comm...
File components/policy/core/common/cloud/enterprise_metrics.h (right):

https://codereview.chromium.org/2959853003/diff/1/components/policy/core/comm...
components/policy/core/common/cloud/enterprise_metrics.h:233:
kEnrollmentLicenseRequestFailed = 57,
On 2017/07/07 14:32:34, emaxx wrote:
> nit: s/kEnrollmentLicenseRequestFailed/kMetricEnrollmentLicenseRequestFailed/
> (following the style of the previous 56 items of the enum)

Done.

[Denis Kuznetsov (DE-MUC), 2017-07-13 18:44:12]

The CQ bit was unchecked by antrim@chromium.org

[Denis Kuznetsov (DE-MUC), 2017-07-13 18:46:51]

The CQ bit was checked by antrim@chromium.org

[Denis Kuznetsov (DE-MUC), 2017-07-13 18:46:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
alemate@chromium.org, nkostylev@chromium.org, emaxx@chromium.org
Link to the patchset: https://codereview.chromium.org/2959853003/#ps40001
(title: "Addressed comments")

[commit-bot: I haz the power, 2017-07-13 18:47:08]

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, 2017-07-13 20:58:45]

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

[commit-bot: I haz the power, 2017-07-13 20:58:47]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Denis Kuznetsov (DE-MUC), 2017-07-25 19:00:15]

The CQ bit was checked by antrim@chromium.org

[Denis Kuznetsov (DE-MUC), 2017-07-25 19:00:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
alemate@chromium.org, nkostylev@chromium.org, emaxx@chromium.org
Link to the patchset: https://codereview.chromium.org/2959853003/#ps60001
(title: "Rebase to ToT")

[commit-bot: I haz the power, 2017-07-25 19:00:33]

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, 2017-07-25 20:42:26]

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

[commit-bot: I haz the power, 2017-07-25 20:42:27]

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

[Denis Kuznetsov (DE-MUC), 2017-07-25 21:55:01]

The CQ bit was checked by antrim@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-25 21:55:11]

Dry run: CQ is trying da patch.

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

[Denis Kuznetsov (DE-MUC), 2017-07-25 22:43:46]

The CQ bit was checked by antrim@chromium.org

[Denis Kuznetsov (DE-MUC), 2017-07-25 22:43:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
alemate@chromium.org, nkostylev@chromium.org, emaxx@chromium.org
Link to the patchset: https://codereview.chromium.org/2959853003/#ps80001
(title: "Remove unused flex import")

[commit-bot: I haz the power, 2017-07-25 22:44:05]

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, 2017-07-26 02:15:06]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1501022626242660,
"parent_rev": "cc048f893dac3da0376abf7adc781b0452691a0d", "commit_rev":
"3d1e27d07f9eb13b298556a877d4b54bcb94ca33"}

[commit-bot: I haz the power, 2017-07-26 02:15:38]

Description was changed from

==========
Add UI components for license type selection in ChromeOS login UI.
Changes that would actually use them (enrollment screen) would come in next CL.

BUG=725124
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Add UI components for license type selection in ChromeOS login UI.
Changes that would actually use them (enrollment screen) would come in next CL.

BUG=725124
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2959853003
Cr-Commit-Position: refs/heads/master@{#489529}
Committed:
https://chromium.googlesource.com/chromium/src/+/3d1e27d07f9eb13b298556a877d4...
==========

[commit-bot: I haz the power, 2017-07-26 02:15:39]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/3d1e27d07f9eb13b298556a877d4...