[Chromecast] Aggregate errors in multizone backend test.

[Chromecast] Aggregate errors in multizone backend test.

Use only one effects stream.
Use PCM format supported by AMP.

BUG=internal b/27694837
TEST=cast_multizone_backend_unittests

Committed: https://crrev.com/a7c82456f81ecf71bceaa36cf31841b324556ded
Cr-Commit-Position: refs/heads/master@{#400067}

[jameswest, 2016-06-15 20:17:23]

Patchset #2 (id:20001) has been deleted

[jameswest, 2016-06-15 20:19:08]

Description was changed from

==========
[Chromecast] Aggregate errors in multizone backend test.

BUG=internal b/27694837
TEST=cast_multizone_backend_unittests
==========

to

==========
[Chromecast] Aggregate errors in multizone backend test.

Use only one effects stream.
Use PCM format supported by AMP.

BUG=internal b/27694837
TEST=cast_multizone_backend_unittests
==========

[jameswest, 2016-06-15 20:20:25]

jameswest@google.com changed reviewers:
+ kmackay@chromium.org, slan@chromium.org

[kmackay, 2016-06-15 20:26:48]

https://codereview.chromium.org/2074473002/diff/1/chromecast/media/cma/backen...
File chromecast/media/cma/backend/multizone_backend_unittest.cc (right):

https://codereview.chromium.org/2074473002/diff/1/chromecast/media/cma/backen...
chromecast/media/cma/backend/multizone_backend_unittest.cc:260:
average_rendering_delay_error_us_ =
Seems like it would be more accurate to maintain the total render delay error
(sum of abs(error)), and then compute the average at the end

[jameswest, 2016-06-15 21:41:01]

https://codereview.chromium.org/2074473002/diff/1/chromecast/media/cma/backen...
File chromecast/media/cma/backend/multizone_backend_unittest.cc (right):

https://codereview.chromium.org/2074473002/diff/1/chromecast/media/cma/backen...
chromecast/media/cma/backend/multizone_backend_unittest.cc:260:
average_rendering_delay_error_us_ =
On 2016/06/15 20:26:48, kmackay wrote:
> Seems like it would be more accurate to maintain the total render delay error
> (sum of abs(error)), and then compute the average at the end

Done.

[kmackay, 2016-06-15 22:52:10]

lgtm

[slan, 2016-06-15 22:53:13]

LGTM

[jameswest, 2016-06-16 00:04:16]

The CQ bit was checked by jameswest@google.com

[commit-bot: I haz the power, 2016-06-16 00:05:10]

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

[commit-bot: I haz the power, 2016-06-16 01:18:15]

Description was changed from

==========
[Chromecast] Aggregate errors in multizone backend test.

Use only one effects stream.
Use PCM format supported by AMP.

BUG=internal b/27694837
TEST=cast_multizone_backend_unittests
==========

to

==========
[Chromecast] Aggregate errors in multizone backend test.

Use only one effects stream.
Use PCM format supported by AMP.

BUG=internal b/27694837
TEST=cast_multizone_backend_unittests
==========

[commit-bot: I haz the power, 2016-06-16 01:18:16]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-06-16 01:18:18]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-16 01:19:55]

Description was changed from

==========
[Chromecast] Aggregate errors in multizone backend test.

Use only one effects stream.
Use PCM format supported by AMP.

BUG=internal b/27694837
TEST=cast_multizone_backend_unittests
==========

to

==========
[Chromecast] Aggregate errors in multizone backend test.

Use only one effects stream.
Use PCM format supported by AMP.

BUG=internal b/27694837
TEST=cast_multizone_backend_unittests

Committed: https://crrev.com/a7c82456f81ecf71bceaa36cf31841b324556ded
Cr-Commit-Position: refs/heads/master@{#400067}
==========

[commit-bot: I haz the power, 2016-06-16 01:19:57]

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