Recommit and fix of "Added a ForceYouTubeRestrict policy and deprecated the old ForceYouTubeSafetyM…

Recommit and fix of "Added a ForceYouTubeRestrict policy and deprecated the old ForceYouTubeSafetyMode policy"
Git hash: c42c0dddcb3c630cb30096d16a4485bd1025eccc
Link: https://codereview.chromium.org/2239753002/

The CL was reverted because of a timeout in the PolicyTest.ForceSafeSearch browser test. The test was doing a lot of calls to the slow CheckSafeSearch function. To fix this, the browser test is split into two separate tests:
LegacySafeSearch verifies that prefs are set properly.
ForceGoogleSafeSearch verifies that safe search actually is in effect depending on the pref.

TBR=tnagel@chromium.org
BUG=590478
Committed: https://crrev.com/98684cb7910faea91679c00a54484237e386a099
Cr-Commit-Position: refs/heads/master@{#424413}

[ljusten (tachyonic), 2016-10-07 12:16:10]

Description was changed from

==========
Recommit and fix of "Added a ForceYouTubeRestrict policy and deprecated the old
ForceYouTubeSafetyMode policy"
Git hash: c42c0dddcb3c630cb30096d16a4485bd1025eccc

TBR=tnagel@chromium.org
BUG=590478
==========

to

==========
Recommit and fix of "Added a ForceYouTubeRestrict policy and deprecated the old
ForceYouTubeSafetyMode policy"
Git hash: c42c0dddcb3c630cb30096d16a4485bd1025eccc
Link: https://codereview.chromium.org/2239753002/

The CL was reverted because of a timeout in the PolicyTest.ForceSafeSearch
browser test. The test was doing a lot of calls to the slow CheckSafeSearch
function. To fix this, the browser test is split into two separate tests:
LegacySafeSearch verifies that prefs are set properly.
ForceGoogleSafeSearch verifies that safe search actually is in effect depending
on the pref.

TBR=tnagel@chromium.org
BUG=590478
==========

[ljusten (tachyonic), 2016-10-07 12:34:54]

Thiemo, PTAL

[Thiemo Nagel, 2016-10-10 14:11:27]

lgtm

[ljusten (tachyonic), 2016-10-11 10:42:21]

The CQ bit was checked by ljusten@chromium.org

[ljusten (tachyonic), 2016-10-11 10:42:21]

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

[commit-bot: I haz the power, 2016-10-11 10:42:29]

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, 2016-10-11 10:50:58]

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

[commit-bot: I haz the power, 2016-10-11 10:50:59]

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...)

[ljusten (tachyonic), 2016-10-11 11:59:41]

The CQ bit was checked by ljusten@chromium.org

[commit-bot: I haz the power, 2016-10-11 11:59:47]

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, 2016-10-11 12:08:48]

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

[commit-bot: I haz the power, 2016-10-11 12:08:49]

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...)

[ljusten (tachyonic), 2016-10-11 13:14:31]

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

[commit-bot: I haz the power, 2016-10-11 13:14: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, 2016-10-11 13:19:31]

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

[commit-bot: I haz the power, 2016-10-11 13:19:31]

Dry run: This issue passed the CQ dry run.

[ljusten (tachyonic), 2016-10-11 13:20:50]

The CQ bit was checked by ljusten@chromium.org

[commit-bot: I haz the power, 2016-10-11 13:21:01]

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, 2016-10-11 13:26:49]

Description was changed from

==========
Recommit and fix of "Added a ForceYouTubeRestrict policy and deprecated the old
ForceYouTubeSafetyMode policy"
Git hash: c42c0dddcb3c630cb30096d16a4485bd1025eccc
Link: https://codereview.chromium.org/2239753002/

The CL was reverted because of a timeout in the PolicyTest.ForceSafeSearch
browser test. The test was doing a lot of calls to the slow CheckSafeSearch
function. To fix this, the browser test is split into two separate tests:
LegacySafeSearch verifies that prefs are set properly.
ForceGoogleSafeSearch verifies that safe search actually is in effect depending
on the pref.

TBR=tnagel@chromium.org
BUG=590478
==========

to

==========
Recommit and fix of "Added a ForceYouTubeRestrict policy and deprecated the old
ForceYouTubeSafetyMode policy"
Git hash: c42c0dddcb3c630cb30096d16a4485bd1025eccc
Link: https://codereview.chromium.org/2239753002/

The CL was reverted because of a timeout in the PolicyTest.ForceSafeSearch
browser test. The test was doing a lot of calls to the slow CheckSafeSearch
function. To fix this, the browser test is split into two separate tests:
LegacySafeSearch verifies that prefs are set properly.
ForceGoogleSafeSearch verifies that safe search actually is in effect depending
on the pref.

TBR=tnagel@chromium.org
BUG=590478
==========

[commit-bot: I haz the power, 2016-10-11 13:26:50]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-11 13:29:26]

Description was changed from

==========
Recommit and fix of "Added a ForceYouTubeRestrict policy and deprecated the old
ForceYouTubeSafetyMode policy"
Git hash: c42c0dddcb3c630cb30096d16a4485bd1025eccc
Link: https://codereview.chromium.org/2239753002/

The CL was reverted because of a timeout in the PolicyTest.ForceSafeSearch
browser test. The test was doing a lot of calls to the slow CheckSafeSearch
function. To fix this, the browser test is split into two separate tests:
LegacySafeSearch verifies that prefs are set properly.
ForceGoogleSafeSearch verifies that safe search actually is in effect depending
on the pref.

TBR=tnagel@chromium.org
BUG=590478
==========

to

==========
Recommit and fix of "Added a ForceYouTubeRestrict policy and deprecated the old
ForceYouTubeSafetyMode policy"
Git hash: c42c0dddcb3c630cb30096d16a4485bd1025eccc
Link: https://codereview.chromium.org/2239753002/

The CL was reverted because of a timeout in the PolicyTest.ForceSafeSearch
browser test. The test was doing a lot of calls to the slow CheckSafeSearch
function. To fix this, the browser test is split into two separate tests:
LegacySafeSearch verifies that prefs are set properly.
ForceGoogleSafeSearch verifies that safe search actually is in effect depending
on the pref.

TBR=tnagel@chromium.org
BUG=590478

Committed: https://crrev.com/98684cb7910faea91679c00a54484237e386a099
Cr-Commit-Position: refs/heads/master@{#424413}
==========

[commit-bot: I haz the power, 2016-10-11 13:29:28]

Patchset 3 (id:??) landed as
https://crrev.com/98684cb7910faea91679c00a54484237e386a099
Cr-Commit-Position: refs/heads/master@{#424413}