Add finch control for use-zoom-for-dsf.

Add finch control for use-zoom-for-dsf.

BUG=613414
Committed: https://crrev.com/9dc180d31f8452c215651ccacbf1b5fc731f479d
Cr-Commit-Position: refs/heads/master@{#405922}

[Bret, 2016-07-14 21:30:21]

Description was changed from

==========
Add finch control for use-zoom-for-dsf.

BUG=613414
==========

to

==========
Add finch control for use-zoom-for-dsf.

BUG=613414
==========

[Bret, 2016-07-14 21:30:21]

bsep@chromium.org changed reviewers:
+ jam@chromium.org, wfh@chromium.org

[Will Harris, 2016-07-14 22:42:56]

lgtm fwiw

https://codereview.chromium.org/2147143003/diff/1/content/common/content_swit...
File content/common/content_switches_internal.cc (right):

https://codereview.chromium.org/2147143003/diff/1/content/common/content_swit...
content/common/content_switches_internal.cc:23: const base::Feature
kUseZoomForDsfEnabledByDefault {
I use constexpr here but I think it's the same difference.

[Bret, 2016-07-15 18:12:45]

bsep@chromium.org changed reviewers:
+ nasko@chromium.org
- jam@chromium.org

[Bret, 2016-07-15 18:14:28]

Changing reviewers because John is ooo.

https://codereview.chromium.org/2147143003/diff/1/content/common/content_swit...
File content/common/content_switches_internal.cc (right):

https://codereview.chromium.org/2147143003/diff/1/content/common/content_swit...
content/common/content_switches_internal.cc:23: const base::Feature
kUseZoomForDsfEnabledByDefault {
On 2016/07/14 22:42:56, Will Harris wrote:
> I use constexpr here but I think it's the same difference.

The example used const and this seems to work fine so I think I'll leave it.

[nasko, 2016-07-15 23:50:14]

Rubberstamp LGTM

[Bret, 2016-07-16 00:01:01]

The CQ bit was checked by bsep@chromium.org

[commit-bot: I haz the power, 2016-07-16 00:02:23]

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-07-16 00:09:58]

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

[commit-bot: I haz the power, 2016-07-16 00:09:59]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[Bret, 2016-07-16 00:32:58]

The CQ bit was checked by bsep@chromium.org

[Bret, 2016-07-16 00:32:58]

The patchset sent to the CQ was uploaded after l-g-t-m from nasko@chromium.org,
wfh@chromium.org
Link to the patchset: https://codereview.chromium.org/2147143003/#ps20001
(title: "only compile feature on windows")

[commit-bot: I haz the power, 2016-07-16 00:33:19]

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-07-16 02:22:37]

Description was changed from

==========
Add finch control for use-zoom-for-dsf.

BUG=613414
==========

to

==========
Add finch control for use-zoom-for-dsf.

BUG=613414
==========

[commit-bot: I haz the power, 2016-07-16 02:22:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-16 02:24:45]

Description was changed from

==========
Add finch control for use-zoom-for-dsf.

BUG=613414
==========

to

==========
Add finch control for use-zoom-for-dsf.

BUG=613414

Committed: https://crrev.com/9dc180d31f8452c215651ccacbf1b5fc731f479d
Cr-Commit-Position: refs/heads/master@{#405922}
==========

[commit-bot: I haz the power, 2016-07-16 02:24:46]

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