Only look at the last 10K samples for WebRTC muting tests.

Only look at the last 10K samples for WebRTC muting tests.

Chrome has maybe gotten a bit slower at muting, but it's at most 1 sec.
This test should not be a performance test of muting speed though,
merely that muting happens, so update the test to just look at the last
10K samples in case we're unlucky and get a beep into the samples
window before the mute kicks in.

BUG=648879
R=ehmaldonado@chromium.org

Committed: https://crrev.com/4589e9a6c8fa01dab372c1b35a149ceaed8215f1
Cr-Commit-Position: refs/heads/master@{#421139}

[ehmaldonado_chromium, 2016-09-26 09:28:51]

On 2016/09/26 09:27:05, phoglund_chrome_000_sep26 wrote:

LGTM
What was the number of samples before?

[phoglund_chromium, 2016-09-26 12:56:24]

On 2016/09/26 09:28:51, ehmaldonado_chromium wrote:
> On 2016/09/26 09:27:05, phoglund_chrome_000_sep26 wrote:
> 
> LGTM
> What was the number of samples before?

Depends on how the long the test was looking for silence, but in the bug I
observed 130K samples. That in itself was the problem: some of the early samples
in there had high values which made the average take long to get low enough.

[phoglund_chromium, 2016-09-26 12:56:27]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2016-09-26 12:56:42]

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-09-26 15:25:19]

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

[commit-bot: I haz the power, 2016-09-26 15:25:20]

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

[phoglund_chromium, 2016-09-26 15:27:16]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2016-09-26 15:27:48]

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-09-26 19:21:20]

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

[commit-bot: I haz the power, 2016-09-26 19:21:21]

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

[phoglund_chromium, 2016-09-27 07:13:38]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2016-09-27 07:13:49]

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-09-27 07:46:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 07:48:44]

Description was changed from

==========
Only look at the last 10K samples for WebRTC muting tests.

Chrome has maybe gotten a bit slower at muting, but it's at most 1 sec.
This test should not be a performance test of muting speed though,
merely that muting happens, so update the test to just look at the last
10K samples in case we're unlucky and get a beep into the samples
window before the mute kicks in.

BUG=648879
R=ehmaldonado@chromium.org
==========

to

==========
Only look at the last 10K samples for WebRTC muting tests.

Chrome has maybe gotten a bit slower at muting, but it's at most 1 sec.
This test should not be a performance test of muting speed though,
merely that muting happens, so update the test to just look at the last
10K samples in case we're unlucky and get a beep into the samples
window before the mute kicks in.

BUG=648879
R=ehmaldonado@chromium.org

Committed: https://crrev.com/4589e9a6c8fa01dab372c1b35a149ceaed8215f1
Cr-Commit-Position: refs/heads/master@{#421139}
==========

[commit-bot: I haz the power, 2016-09-27 07:48:45]

Patchset 1 (id:??) landed as
https://crrev.com/4589e9a6c8fa01dab372c1b35a149ceaed8215f1
Cr-Commit-Position: refs/heads/master@{#421139}