Restart the Chrome session after a successful KIOSK enrollment.

Restart the Chrome session after a successful KIOSK enrollment.

BUG=chromium-os:28586
TEST=Enroll in KIOSK mode and ensure the session restarts when enrollment is done.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=129396

[pastarmovj, 2012-03-28 11:20:26]

Can you bless this one please?

[Joao da Silva, 2012-03-28 11:36:37]

lgtm

http://codereview.chromium.org/9874017/diff/1/chrome/browser/chromeos/login/e...
File chrome/browser/chromeos/login/enrollment/enterprise_enrollment_screen.cc
(right):

http://codereview.chromium.org/9874017/diff/1/chrome/browser/chromeos/login/e...
chrome/browser/chromeos/login/enrollment/enterprise_enrollment_screen.cc:86:
GetDeviceCloudPolicyDataStore()->device_mode() ==
How about just browser_policy_connector()->GetDeviceMode()?

I'm just not sure the EnterpriseInstallAttributes will get the right values
right after enrollment.

[pastarmovj, 2012-03-28 11:43:15]

http://codereview.chromium.org/9874017/diff/1/chrome/browser/chromeos/login/e...
File chrome/browser/chromeos/login/enrollment/enterprise_enrollment_screen.cc
(right):

http://codereview.chromium.org/9874017/diff/1/chrome/browser/chromeos/login/e...
chrome/browser/chromeos/login/enrollment/enterprise_enrollment_screen.cc:86:
GetDeviceCloudPolicyDataStore()->device_mode() ==
On 2012/03/28 11:36:37, Joao da Silva wrote:
> How about just browser_policy_connector()->GetDeviceMode()?
> 
> I'm just not sure the EnterpriseInstallAttributes will get the right values
> right after enrollment.

They must have the right value if registration has succeeded or else we will not
fetch policy. The race Mattias is working on is much earlier in the flow. At
this point in code we discuss here we are in very deep horse-dong if the
InstallAttribs are not yet in sync :)