Add a "device disabled" OOBE screen

Add a "device disabled" OOBE screen

This screen will be shown when the owner/administrator remotely disables
a device. The CL only adds the screen. A follow-up CL will hook it up
with the OOBE flow.

The UI and strings are still under review and may change slightly before
this feature launches.

BUG=425574
TEST=Manual

Committed: https://crrev.com/2edfb70f1ecf64d7fd487047364f784eea23a8ea
Cr-Commit-Position: refs/heads/master@{#300731}

[bartfab (slow), 2014-10-22 09:52:06]

bartfab@chromium.org changed reviewers:
+ nkostylev@chromium.org

[bartfab (slow), 2014-10-22 09:52:07]

Hi Nikita,

Could you take a look please?

[Nikita (slow), 2014-10-22 10:28:25]

Please add link to screenshots (Drive folder for instance).

[Nikita (slow), 2014-10-22 10:33:53]

lgtm

https://codereview.chromium.org/669623006/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/login/screen_device_disabled.html
(right):

https://codereview.chromium.org/669623006/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/login/screen_device_disabled.html:9:
i18n-content="deviceDisabledExplanation">
nit: 4 spaces indent

https://codereview.chromium.org/669623006/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/login/screen_device_disabled.js (right):

https://codereview.chromium.org/669623006/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/login/screen_device_disabled.js:28:
setMessage: function(message) {
This will be used in next CL?

https://codereview.chromium.org/669623006/diff/1/chrome/browser/ui/webui/chro...
File chrome/browser/ui/webui/chromeos/login/device_disabled_screen_handler.cc
(right):

https://codereview.chromium.org/669623006/diff/1/chrome/browser/ui/webui/chro...
chrome/browser/ui/webui/chromeos/login/device_disabled_screen_handler.cc:40: }
nit: show_on_init_ = false;

[bartfab (slow), 2014-10-22 11:32:16]

I posted a screen shot on the bug report:

http://crbug.com/425574#c2

https://codereview.chromium.org/669623006/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/login/screen_device_disabled.html
(right):

https://codereview.chromium.org/669623006/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/login/screen_device_disabled.html:9:
i18n-content="deviceDisabledExplanation">
On 2014/10/22 10:33:53, Nikita Kostylev wrote:
> nit: 4 spaces indent

Done.

https://codereview.chromium.org/669623006/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/login/screen_device_disabled.js (right):

https://codereview.chromium.org/669623006/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/login/screen_device_disabled.js:28:
setMessage: function(message) {
On 2014/10/22 10:33:53, Nikita Kostylev wrote:
> This will be used in next CL?

Yes.

https://codereview.chromium.org/669623006/diff/1/chrome/browser/ui/webui/chro...
File chrome/browser/ui/webui/chromeos/login/device_disabled_screen_handler.cc
(right):

https://codereview.chromium.org/669623006/diff/1/chrome/browser/ui/webui/chro...
chrome/browser/ui/webui/chromeos/login/device_disabled_screen_handler.cc:40: }
On 2014/10/22 10:33:53, Nikita Kostylev wrote:
> nit: show_on_init_ = false;

Done. Note that none of the other screens do this. It would seem that they all
should.

[bartfab (slow), 2014-10-22 11:32:28]

The CQ bit was checked by bartfab@chromium.org

[commit-bot: I haz the power, 2014-10-22 11:32:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/669623006/20001

[commit-bot: I haz the power, 2014-10-22 11:38:58]

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

[commit-bot: I haz the power, 2014-10-22 11:38:58]

Try jobs failed on following builders:
  linux_chromium_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)

[Nikita (slow), 2014-10-22 11:59:14]

lgtm

https://codereview.chromium.org/669623006/diff/1/chrome/browser/ui/webui/chro...
File chrome/browser/ui/webui/chromeos/login/device_disabled_screen_handler.cc
(right):

https://codereview.chromium.org/669623006/diff/1/chrome/browser/ui/webui/chro...
chrome/browser/ui/webui/chromeos/login/device_disabled_screen_handler.cc:40: }
On 2014/10/22 11:32:16, bartfab wrote:
> On 2014/10/22 10:33:53, Nikita Kostylev wrote:
> > nit: show_on_init_ = false;
> 
> Done. Note that none of the other screens do this. It would seem that they all
> should.

You're right, we'll address that.

[bartfab (slow), 2014-10-22 13:11:13]

The CQ bit was checked by bartfab@chromium.org

[commit-bot: I haz the power, 2014-10-22 13:11:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/669623006/20001

[commit-bot: I haz the power, 2014-10-22 19:28:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-22 19:29:28]

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