multibuffer field trial

multibuffer field trial

BUG=514719
Committed: https://crrev.com/2703d09eeaed647bec80a34af141bd8831378c7c
Cr-Commit-Position: refs/heads/master@{#374773}

[hubbe, 2016-02-05 00:53:34]

hubbe@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[DaleCurtis, 2016-02-05 01:05:18]

https://codereview.chromium.org/1670753003/diff/1/media/blink/webmediaplayer_...
File media/blink/webmediaplayer_impl.cc (right):

https://codereview.chromium.org/1670753003/diff/1/media/blink/webmediaplayer_...
media/blink/webmediaplayer_impl.cc:294: // TODO(hubbe): This experiment is
temporary and should be removed once
I suspect the experiment folk are going to want you to turn this into a feature
flag instead of using the switches::kUseNewMediaCache. go/finch101 has the
details.

[hubbe, 2016-02-09 22:01:59]

https://codereview.chromium.org/1670753003/diff/1/media/blink/webmediaplayer_...
File media/blink/webmediaplayer_impl.cc (right):

https://codereview.chromium.org/1670753003/diff/1/media/blink/webmediaplayer_...
media/blink/webmediaplayer_impl.cc:294: // TODO(hubbe): This experiment is
temporary and should be removed once
On 2016/02/05 01:05:18, DaleCurtis wrote:
> I suspect the experiment folk are going to want you to turn this into a
feature
> flag instead of using the switches::kUseNewMediaCache. go/finch101 has the
> details.

Better now?

[DaleCurtis, 2016-02-09 22:48:34]

dalecurtis@chromium.org changed reviewers:
+ asvitkine@chromium.org

[DaleCurtis, 2016-02-09 22:48:34]

lgtm, +asvitkine to double check feature usage.

[Alexei Svitkine (slow), 2016-02-10 17:29:50]

lgtm

https://codereview.chromium.org/1670753003/diff/40001/media/base/media_switch...
File media/base/media_switches.h (right):

https://codereview.chromium.org/1670753003/diff/40001/media/base/media_switch...
media/base/media_switches.h:68: namespace media {
Nit: Add an empty line after this.

[hubbe, 2016-02-10 18:07:12]

The CQ bit was checked by hubbe@chromium.org

[hubbe, 2016-02-10 18:07:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
asvitkine@chromium.org, dalecurtis@chromium.org
Link to the patchset: https://codereview.chromium.org/1670753003/#ps60001
(title: "added newline")

[commit-bot: I haz the power, 2016-02-10 18:07:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1670753003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1670753003/60001

[commit-bot: I haz the power, 2016-02-10 21:23:27]

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

[commit-bot: I haz the power, 2016-02-10 21:23:27]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[hubbe, 2016-02-10 21:53:39]

The CQ bit was checked by hubbe@chromium.org

[commit-bot: I haz the power, 2016-02-10 21:56:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1670753003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1670753003/60001

[hubbe, 2016-02-10 22:10:18]

https://codereview.chromium.org/1670753003/diff/40001/media/base/media_switch...
File media/base/media_switches.h (right):

https://codereview.chromium.org/1670753003/diff/40001/media/base/media_switch...
media/base/media_switches.h:68: namespace media {
On 2016/02/10 17:29:50, Alexei Svitkine wrote:
> Nit: Add an empty line after this.

Done.

[commit-bot: I haz the power, 2016-02-10 23:17:24]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-02-16 22:32:19]

Description was changed from

==========
multibuffer field trial

BUG=514719
==========

to

==========
multibuffer field trial

BUG=514719

Committed: https://crrev.com/2703d09eeaed647bec80a34af141bd8831378c7c
Cr-Commit-Position: refs/heads/master@{#374773}
==========

[commit-bot: I haz the power, 2016-02-16 22:32:19]

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