Rename enterprise policy proto fields from KIOSK to RETAIL.

Rename enterprise policy proto fields from KIOSK to RETAIL.

R=mnissler@chromium.org
BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=128226

[pastarmovj, 2012-03-21 15:37:58]

Wow I just used the R flag and it added you as a reviewer automagically. But
because I don't know if you will get notified with a mail I send you one
explicitly.

[Bin, 2012-03-21 16:17:31]

http://codereview.chromium.org/9814011/diff/1/chrome/browser/policy/proto/chr...
File chrome/browser/policy/proto/chrome_device_policy.proto (left):

http://codereview.chromium.org/9814011/diff/1/chrome/browser/policy/proto/chr...
chrome/browser/policy/proto/chrome_device_policy.proto:163: message
StartUpUrlsProto {
This is newly added setting. I need to add it on the server-side.

http://codereview.chromium.org/9814011/diff/1/chrome/browser/policy/proto/dev...
File chrome/browser/policy/proto/device_management_backend.proto (right):

http://codereview.chromium.org/9814011/diff/1/chrome/browser/policy/proto/dev...
chrome/browser/policy/proto/device_management_backend.proto:45: optional bool
known_machine_id = 5;
Can you change this name to auto_enrolled?

Belows is what's on the server-side.

// This is used for the client to indicate that the device is doing 
// registration as part of auto enrollment process.
optional bool auto_enrolled = 5;

[Mattias Nissler (ping if slow), 2012-03-21 16:35:32]

Look, it is RETAIL! No, it's a KIOSK! RETAIL! KIOSK!

I can see us switching back in the future when we do the proper KIOSK
implementation, for now I just don't care.

Please address Bin's comment, after that, LGTM.

[commit-bot: I haz the power, 2012-03-21 16:59:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pastarmovj@chromium.org/9814011/5001

[commit-bot: I haz the power, 2012-03-21 17:21:47]

Try job failure for 9814011-5001 (retry) on linux_chromeos for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[Mattias Nissler (ping if slow), 2012-03-22 10:42:46]

LGTM provided you get green tryjobs

[commit-bot: I haz the power, 2012-03-22 11:50:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pastarmovj@chromium.org/9814011/8007

[commit-bot: I haz the power, 2012-03-22 11:50:57]

Presubmit check for 9814011-8007 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    net/tools/testserver

Presubmit checks took 1.1s to calculate.

[pastarmovj, 2012-03-22 12:03:11]

Adding phajdan.jr as the CL needs owner approval for the changes in
net/tools/testserver.

[Paweł Hajdan Jr., 2012-03-22 14:07:40]

testserver LGTM

[commit-bot: I haz the power, 2012-03-22 15:21:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pastarmovj@chromium.org/9814011/8007

[commit-bot: I haz the power, 2012-03-22 16:35:07]

Change committed as 128226