Enable MediaFoundationH264Encoding by default

Enable MediaFoundationH264Encoding by default

This CL enables this feature by default after the finch experiment.

BUG=631783
Review-Url: https://codereview.chromium.org/2714153004
Cr-Commit-Position: refs/heads/master@{#452948}
Committed: https://chromium.googlesource.com/chromium/src/+/2f52fcb3a305091ded42e8fe8ed5a0c0efe03013

[emircan, 2017-02-24 18:43:47]

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

[commit-bot: I haz the power, 2017-02-24 18:44:18]

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

[emircan, 2017-02-24 18:45:28]

Description was changed from

==========
enable.

BUG=
==========

to

==========
Enable MediaFoundationH264Encoding by default

This CL enables this feature by default after the finch experiment.

BUG=631783
==========

[emircan, 2017-02-24 18:45:28]

emircan@chromium.org changed reviewers:
+ rkaplow@chromium.org, sdoyon@chromium.org

[emircan, 2017-02-24 18:46:04]

PTAL.

[rkaplow, 2017-02-24 18:59:00]

lgtm

lgtm but probably should be reviewed by someone on the feature team. 

Don't forget to cleanup the experiment code at this time.

[commit-bot: I haz the power, 2017-02-24 19:10:25]

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

[commit-bot: I haz the power, 2017-02-24 19:10:26]

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

[emircan, 2017-02-24 21:09:33]

On 2017/02/24 18:59:00, rkaplow (slow) wrote:
> lgtm
> 
> lgtm but probably should be reviewed by someone on the feature team. 
> 
> Don't forget to cleanup the experiment code at this time.

Thanks. I added sdoyon@ to take a look as well. There is no unused experiment
code right now and I want to keep the feature flag there as a kill switch in
case of a problem.

[sdoyon, 2017-02-24 21:26:11]

LGTM

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[emircan, 2017-02-24 21:35:39]

emircan@chromium.org changed reviewers:
+ sandersd@chromium.org

[emircan, 2017-02-24 21:35:40]

Thanks. 

sandersd@ RS review please?

[sandersd (OOO until July 31), 2017-02-24 21:58:13]

lgtm

[emircan, 2017-02-24 22:11:15]

The CQ bit was checked by emircan@chromium.org

[commit-bot: I haz the power, 2017-02-24 22:12:16]

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-02-24 22:17:44]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487974275671220, "parent_rev":
"a534bdcb6f2f02960bfd7bab54c0856403563281", "commit_rev":
"2f52fcb3a305091ded42e8fe8ed5a0c0efe03013"}

[commit-bot: I haz the power, 2017-02-24 22:18:19]

Description was changed from

==========
Enable MediaFoundationH264Encoding by default

This CL enables this feature by default after the finch experiment.

BUG=631783
==========

to

==========
Enable MediaFoundationH264Encoding by default

This CL enables this feature by default after the finch experiment.

BUG=631783

Review-Url: https://codereview.chromium.org/2714153004
Cr-Commit-Position: refs/heads/master@{#452948}
Committed:
https://chromium.googlesource.com/chromium/src/+/2f52fcb3a305091ded42e8fe8ed5...
==========

[commit-bot: I haz the power, 2017-02-24 22:18:20]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/2f52fcb3a305091ded42e8fe8ed5...