Restart Chrome if active kiosk app has been removed from policy.

Restart Chrome if active kiosk app has been removed from policy.

BUG=439285
Committed: https://crrev.com/d8d00377c60669198d2a58eade51ae8edd15fc6d
Cr-Commit-Position: refs/heads/master@{#318986}

[Vladislav Kaznacheev, 2015-03-03 23:34:46]

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

[Vladislav Kaznacheev, 2015-03-03 23:34:47]

Hi Xiyuan,

Please see if this make sense. This takes care of first half of the problem:
restarting Chrome once the app is no longer in the policy. I plan to address the
removal of the currently mounted cryptohome in a separate CL.

Thanks!

  Vlad

[xiyuan, 2015-03-03 23:59:59]

lgtm

Works for me.

https://chromiumcodereview.appspot.com/975193002/diff/1/chrome/browser/chrome...
File chrome/browser/chromeos/app_mode/kiosk_app_manager.cc (right):

https://chromiumcodereview.appspot.com/975193002/diff/1/chrome/browser/chrome...
chrome/browser/chromeos/app_mode/kiosk_app_manager.cc:554: it != old_apps.end();
++it) {
nit: C++11 style to save some typing.

for (const auto& it : old_apps) {
  if (it.second->user_id() == active_user_id) {
...
}

[Vladislav Kaznacheev, 2015-03-04 00:24:35]

The CQ bit was checked by kaznacheev@chromium.org

[Vladislav Kaznacheev, 2015-03-04 00:24:36]

The patchset sent to the CQ was uploaded after l-g-t-m from xiyuan@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/975193002/#ps20001
(title: "Addressed comments")

[Vladislav Kaznacheev, 2015-03-04 00:24:54]

https://chromiumcodereview.appspot.com/975193002/diff/1/chrome/browser/chrome...
File chrome/browser/chromeos/app_mode/kiosk_app_manager.cc (right):

https://chromiumcodereview.appspot.com/975193002/diff/1/chrome/browser/chrome...
chrome/browser/chromeos/app_mode/kiosk_app_manager.cc:554: it != old_apps.end();
++it) {
On 2015/03/03 23:59:59, xiyuan wrote:
> nit: C++11 style to save some typing.
> 
> for (const auto& it : old_apps) {
>   if (it.second->user_id() == active_user_id) {
> ...
> }

Done.

[commit-bot: I haz the power, 2015-03-04 00:26:06]

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

[commit-bot: I haz the power, 2015-03-04 00:57:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-04 00:58:17]

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