Make login screen apps enabled by default

Make login screen apps enabled by default

Invert the command line guard flag, so that the apps on the login screen
are enabled by default now, but can be disabled using the
--disable-login-screen-apps flag. (The flag will be removed after some
time once this change is tested and verified.)

Also open up the corresponding policy DeviceLoginScreenAppInstallList,
with updating its supported range to M60+.

The policy description is extended to mention the security and privacy
constraints that are imposed on it.

BUG=576464
Review-Url: https://codereview.chromium.org/2801113002
Cr-Commit-Position: refs/heads/master@{#471010}
Committed: https://chromium.googlesource.com/chromium/src/+/b4c388b2af93a40a40ebccd3a448c309384b97c1

[emaxx, 2017-04-06 17:07:12]

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

[commit-bot: I haz the power, 2017-04-06 17:08:04]

Dry run: 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-04-06 19:04:47]

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

[commit-bot: I haz the power, 2017-04-06 19:04:48]

Dry run: 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_...)

[emaxx, 2017-04-07 12:04:34]

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

[commit-bot: I haz the power, 2017-04-07 12:04:46]

Dry run: 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-04-07 12:54:52]

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

[commit-bot: I haz the power, 2017-04-07 12:54:52]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[emaxx, 2017-04-07 13:31:55]

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

[commit-bot: I haz the power, 2017-04-07 13:32:25]

Dry run: 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-04-07 13:33:47]

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

[commit-bot: I haz the power, 2017-04-07 13:33:48]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[emaxx, 2017-04-07 13:34:41]

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

[commit-bot: I haz the power, 2017-04-07 13:34:53]

Dry run: 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-04-07 13:35:44]

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

[commit-bot: I haz the power, 2017-04-07 13:35:45]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[emaxx, 2017-04-07 13:47:31]

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

[commit-bot: I haz the power, 2017-04-07 13:47:41]

Dry run: 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-04-07 13:48:34]

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

[commit-bot: I haz the power, 2017-04-07 13:48:35]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[emaxx, 2017-04-07 14:11:11]

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

[commit-bot: I haz the power, 2017-04-07 14:11:16]

Dry run: 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-04-07 14:12:08]

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

[commit-bot: I haz the power, 2017-04-07 14:12:09]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[emaxx, 2017-04-07 14:15:23]

Description was changed from

==========
Make login screen apps enabled by default

Invert the command line guard flag, so that the apps on the login screen
are enabled by default now, but can be disabled using the
--disable-login-screen-apps flag. (The flag will be removed after some
time once this change is tested and verified.)

Also open up the corresponding policy DeviceLoginScreenAppInstallList,
with updating its supported range to M59+.

The policy description is extended to mention the security and privacy
constraints that are imposed on it.

BUG=576464
==========

to

==========
Make login screen apps enabled by default

Invert the command line guard flag, so that the apps on the login screen
are enabled by default now, but can be disabled using the
--disable-login-screen-apps flag. (The flag will be removed after some
time once this change is tested and verified.)

Also open up the corresponding policy DeviceLoginScreenAppInstallList,
with updating its supported range to M59+.

The policy description is extended to mention the security and privacy
constraints that are imposed on it.

BUG=576464
==========

[emaxx, 2017-04-07 15:15:04]

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

[commit-bot: I haz the power, 2017-04-07 15:15:29]

Dry run: 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-04-07 15:55:40]

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

[commit-bot: I haz the power, 2017-04-07 15:55:41]

Dry run: This issue passed the CQ dry run.

[emaxx, 2017-04-11 17:34:35]

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

[commit-bot: I haz the power, 2017-04-11 17:35:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[emaxx, 2017-04-11 17:38:05]

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

[commit-bot: I haz the power, 2017-04-11 17:38:28]

Dry run: 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-04-11 18:13:29]

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

[commit-bot: I haz the power, 2017-04-11 18:13:30]

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

[emaxx, 2017-04-11 22:30:20]

Patchset #6 (id:100001) has been deleted

[emaxx, 2017-04-11 22:30:27]

Patchset #5 (id:80001) has been deleted

[emaxx, 2017-04-11 22:30:56]

Patchset #4 (id:60001) has been deleted

[emaxx, 2017-04-11 22:31:03]

Patchset #3 (id:40001) has been deleted

[emaxx, 2017-04-11 22:31:10]

Patchset #2 (id:20001) has been deleted

[emaxx, 2017-04-11 22:32:04]

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

[commit-bot: I haz the power, 2017-04-11 22:33:52]

Dry run: 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-04-11 23:31:39]

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

[commit-bot: I haz the power, 2017-04-11 23:31:40]

Dry run: This issue passed the CQ dry run.

[emaxx, 2017-04-11 23:37:33]

emaxx@chromium.org changed reviewers:
+ achuith@chromium.org

[emaxx, 2017-04-11 23:37:33]

Achuith: Adding you for a tentative review. I think we shouldn't land this
before the end-to-end browser test is written, but maybe you'd want to provide
some early feedback on this change.

[achuithb, 2017-04-11 23:45:22]

lgtm for 59.

Let's wait for 1 week after M59 branch point to land this, around Apr 20. We
have the flexibility to disable again right before M60 branch point if we're
worried that there's some risk. In the meantime, we may be able to find some
bugs in developer builds or canaries.

[emaxx, 2017-05-09 15:12:55]

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

[emaxx, 2017-05-09 15:13:09]

Description was changed from

==========
Make login screen apps enabled by default

Invert the command line guard flag, so that the apps on the login screen
are enabled by default now, but can be disabled using the
--disable-login-screen-apps flag. (The flag will be removed after some
time once this change is tested and verified.)

Also open up the corresponding policy DeviceLoginScreenAppInstallList,
with updating its supported range to M59+.

The policy description is extended to mention the security and privacy
constraints that are imposed on it.

BUG=576464
==========

to

==========
Make login screen apps enabled by default

Invert the command line guard flag, so that the apps on the login screen
are enabled by default now, but can be disabled using the
--disable-login-screen-apps flag. (The flag will be removed after some
time once this change is tested and verified.)

Also open up the corresponding policy DeviceLoginScreenAppInstallList,
with updating its supported range to M60+.

The policy description is extended to mention the security and privacy
constraints that are imposed on it.

BUG=576464
==========

[commit-bot: I haz the power, 2017-05-09 15:13:39]

Dry run: 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-05-09 17:07:06]

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

[commit-bot: I haz the power, 2017-05-09 17:07:07]

Dry run: 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_...)

[emaxx, 2017-05-09 17:11:23]

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

[commit-bot: I haz the power, 2017-05-09 17:11:55]

Dry run: 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-05-09 18:18:37]

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

[commit-bot: I haz the power, 2017-05-09 18:18:39]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[emaxx, 2017-05-09 21:09:03]

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

[commit-bot: I haz the power, 2017-05-09 21:10:03]

Dry run: 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-05-10 01:26:58]

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

[commit-bot: I haz the power, 2017-05-10 01:26:59]

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

[emaxx, 2017-05-10 21:08:30]

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

[commit-bot: I haz the power, 2017-05-10 21:10:38]

Dry run: 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-05-10 21:44:36]

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

[commit-bot: I haz the power, 2017-05-10 21:44:37]

Dry run: 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_...)

[emaxx, 2017-05-11 01:26:08]

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

[commit-bot: I haz the power, 2017-05-11 01:27:30]

Dry run: CQ is trying da patch.

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

[emaxx, 2017-05-11 03:30:58]

emaxx@chromium.org changed reviewers:
+ finnur@chromium.org, skuhne@chromium.org

[emaxx, 2017-05-11 03:30:59]

+OWNERS
finnur@: Please review changes at chrome/browser/extensions/.
skuhne@: Please review changes at chrome/browser/profiles/profile_manager.cc.
Thanks.

[commit-bot: I haz the power, 2017-05-11 03:38:12]

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

[commit-bot: I haz the power, 2017-05-11 03:38:13]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[emaxx, 2017-05-11 03:48:50]

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

[commit-bot: I haz the power, 2017-05-11 03:50:00]

Dry run: CQ is trying da patch.

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

[Mr4D (OOO till 08-26), 2017-05-11 03:58:38]

profile_manager lgtm

[commit-bot: I haz the power, 2017-05-11 05:44:58]

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

[commit-bot: I haz the power, 2017-05-11 05:44:59]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Finnur, 2017-05-11 11:17:18]

chrome/browser/extensions/ OWNERS LGTM

[emaxx, 2017-05-11 15:16:30]

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

[commit-bot: I haz the power, 2017-05-11 15:16:56]

Dry run: 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-05-11 16:05:27]

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

[commit-bot: I haz the power, 2017-05-11 16:05:28]

Dry run: This issue passed the CQ dry run.

[emaxx, 2017-05-11 18:01:05]

On 2017/05/11 03:58:38, Mr4D (IO delays) wrote:
> profile_manager lgtm
On 2017/05/11 11:17:18, Finnur wrote:
> chrome/browser/extensions/ OWNERS LGTM

Thanks all!

[emaxx, 2017-05-11 18:01:12]

The CQ bit was checked by emaxx@chromium.org

[emaxx, 2017-05-11 18:01:13]

The patchset sent to the CQ was uploaded after l-g-t-m from achuith@chromium.org
Link to the patchset: https://codereview.chromium.org/2801113002/#ps180001
(title: "Rebase")

[commit-bot: I haz the power, 2017-05-11 18:02:57]

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-05-11 18:11:27]

CQ is committing da patch.

Bot data: {"patchset_id": 180001, "attempt_start_ts": 1494525672729500,
"parent_rev": "3b2a27989ad0d93dcefd88c7f63fcb0e893b1c2c", "commit_rev":
"b4c388b2af93a40a40ebccd3a448c309384b97c1"}

[commit-bot: I haz the power, 2017-05-11 18:11:39]

Description was changed from

==========
Make login screen apps enabled by default

Invert the command line guard flag, so that the apps on the login screen
are enabled by default now, but can be disabled using the
--disable-login-screen-apps flag. (The flag will be removed after some
time once this change is tested and verified.)

Also open up the corresponding policy DeviceLoginScreenAppInstallList,
with updating its supported range to M60+.

The policy description is extended to mention the security and privacy
constraints that are imposed on it.

BUG=576464
==========

to

==========
Make login screen apps enabled by default

Invert the command line guard flag, so that the apps on the login screen
are enabled by default now, but can be disabled using the
--disable-login-screen-apps flag. (The flag will be removed after some
time once this change is tested and verified.)

Also open up the corresponding policy DeviceLoginScreenAppInstallList,
with updating its supported range to M60+.

The policy description is extended to mention the security and privacy
constraints that are imposed on it.

BUG=576464

Review-Url: https://codereview.chromium.org/2801113002
Cr-Commit-Position: refs/heads/master@{#471010}
Committed:
https://chromium.googlesource.com/chromium/src/+/b4c388b2af93a40a40ebccd3a448...
==========

[commit-bot: I haz the power, 2017-05-11 18:11:41]

Committed patchset #5 (id:180001) as
https://chromium.googlesource.com/chromium/src/+/b4c388b2af93a40a40ebccd3a448...