Make sure key::kEnableWebBasedSignin policy also turns on command line arg

Make sure key::kEnableWebBasedSignin policy also turns on command line arg
kEnableIframeBasedSignin.  Regression in M41.

BUG=457625
Committed: https://crrev.com/b964c203e990c51d0fc8f5c945312fd6e6455bc2
Cr-Commit-Position: refs/heads/master@{#316262}

[Roger Tawa OOO till Jul 10th, 2015-02-13 15:30:58]

rogerta@chromium.org changed reviewers:
+ atwilson@chromium.org

[Roger Tawa OOO till Jul 10th, 2015-02-13 15:30:59]

Hi Drew,

Please let me know if there is something else to put in the json file to
deprecate this setting, like make the supported_on field or something.  Thanks.

[Roger Tawa OOO till Jul 10th, 2015-02-13 15:31:22]

Patchset #1 (id:1) has been deleted

[Andrew T Wilson (Slow), 2015-02-13 15:38:47]

LGTM with the changes below.

https://codereview.chromium.org/927573002/diff/20001/components/policy/resour...
File components/policy/resources/policy_templates.json (right):

https://codereview.chromium.org/927573002/diff/20001/components/policy/resour...
components/policy/resources/policy_templates.json:1198: 'supported_on':
['chrome.*:35-'],
If the plan is to remove this flag entirely in M42, let's change 'supported_on'
to 'chrome.*:35-41'

https://codereview.chromium.org/927573002/diff/20001/components/policy/resour...
components/policy/resources/policy_templates.json:1209: This setting is no
longer needed with Chrome 41 or later.
Maybe say something like "This setting should no longer be needed as of Chrome
41 and support will be removed entirely in Chrome 42.'

[Roger Tawa OOO till Jul 10th, 2015-02-13 15:54:49]

New patchsets have been uploaded after l-g-t-m from atwilson@chromium.org

[Roger Tawa OOO till Jul 10th, 2015-02-13 15:56:53]

Patchset #2 (id:40001) has been deleted

[Roger Tawa OOO till Jul 10th, 2015-02-13 15:57:46]

Thanks Drew.  As discussed offline, all comments addressed, but I bumped the
version numbers you mentioned by 1 since the removal will happen one version
later.

https://codereview.chromium.org/927573002/diff/20001/components/policy/resour...
File components/policy/resources/policy_templates.json (right):

https://codereview.chromium.org/927573002/diff/20001/components/policy/resour...
components/policy/resources/policy_templates.json:1198: 'supported_on':
['chrome.*:35-'],
On 2015/02/13 15:38:46, Andrew T Wilson wrote:
> If the plan is to remove this flag entirely in M42, let's change
'supported_on'
> to 'chrome.*:35-41'

Done, but used 42 instead of 41.  This will still be available in 42, but with a
different name (rename CL pending).

https://codereview.chromium.org/927573002/diff/20001/components/policy/resour...
components/policy/resources/policy_templates.json:1209: This setting is no
longer needed with Chrome 41 or later.
On 2015/02/13 15:38:46, Andrew T Wilson wrote:
> Maybe say something like "This setting should no longer be needed as of Chrome
> 41 and support will be removed entirely in Chrome 42.'

Done, but used 43 instead of 42.

[Andrew T Wilson (Slow), 2015-02-13 16:10:39]

Still LGTM with new wording. Thanks!

[Roger Tawa OOO till Jul 10th, 2015-02-13 16:11:05]

The CQ bit was checked by rogerta@chromium.org

[commit-bot: I haz the power, 2015-02-13 16:12:21]

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

[Roger Tawa OOO till Jul 10th, 2015-02-13 16:32:02]

The CQ bit was unchecked by rogerta@chromium.org

[Roger Tawa OOO till Jul 10th, 2015-02-13 17:35:04]

New patchsets have been uploaded after l-g-t-m from atwilson@chromium.org

[Roger Tawa OOO till Jul 10th, 2015-02-13 18:36:43]

The CQ bit was checked by rogerta@chromium.org

[commit-bot: I haz the power, 2015-02-13 18:37:37]

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

[commit-bot: I haz the power, 2015-02-13 19:36:21]

Committed patchset #4 (id:100001)

[commit-bot: I haz the power, 2015-02-13 19:36:48]

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