Allow multiple OnMoreData() calls in WASAPIAudioOutputStreamTest.ValidPacketSize

Allow multiple OnMoreData() calls in WASAPIAudioOutputStreamTest.ValidPacketSize

Follow up on https://codereview.chromium.org/1318933003/ and allow
OnMoreData() to be called more than once in the test.

BUG=524947
TEST=WASAPIAudioOutputStreamTest.ValidPacketSize passes

Committed: https://crrev.com/7a7643ee8652ddab8503c57fe0ff1af174ea235b
Cr-Commit-Position: refs/heads/master@{#362439}

[wdzierzanowski, 2015-12-01 08:32:58]

Description was changed from

==========
Allow multiple OnMoreData() calls in WASAPIAudioOutputStreamTest.ValidPacketSize

Follow up on https://codereview.chromium.org/1318933003/ and allow
OnMoreData() to be called more than once in the test.

BUG=524947
TEST=WASAPIAudioOutputStreamTest.ValidPacketSize passes
==========

to

==========
Allow multiple OnMoreData() calls in WASAPIAudioOutputStreamTest.ValidPacketSize

Follow up on https://codereview.chromium.org/1318933003/ and allow
OnMoreData() to be called more than once in the test.

BUG=524947
TEST=WASAPIAudioOutputStreamTest.ValidPacketSize passes
==========

[wdzierzanowski, 2015-12-01 08:32:58]

wdzierzanowski@opera.com changed reviewers:
+ tommi@chromium.org

[wdzierzanowski, 2015-12-01 08:34:07]

PTAL

[tommi (sloooow) - chröme, 2015-12-01 10:06:54]

lgtm. I'll leave it up to you if you want to make the below change.

https://codereview.chromium.org/1487733003/diff/1/media/audio/win/audio_low_l...
File media/audio/win/audio_low_latency_output_win_unittest.cc (right):

https://codereview.chromium.org/1487733003/diff/1/media/audio/win/audio_low_l...
media/audio/win/audio_low_latency_output_win_unittest.cc:385:
.WillRepeatedly(DoAll(QuitLoop(loop.task_runner()),
nit: You could do |WillOnce(<QuitLoop>).WillRepeatedly(<ignore/DoNothing>)| just
to have exactly one quit message.

[wdzierzanowski, 2015-12-01 16:42:40]

https://codereview.chromium.org/1487733003/diff/1/media/audio/win/audio_low_l...
File media/audio/win/audio_low_latency_output_win_unittest.cc (right):

https://codereview.chromium.org/1487733003/diff/1/media/audio/win/audio_low_l...
media/audio/win/audio_low_latency_output_win_unittest.cc:385:
.WillRepeatedly(DoAll(QuitLoop(loop.task_runner()),
On 2015/12/01 10:06:54, tommi wrote:
> nit: You could do |WillOnce(<QuitLoop>).WillRepeatedly(<ignore/DoNothing>)|
just
> to have exactly one quit message.

Done.

[tommi (sloooow) - chröme, 2015-12-01 16:55:01]

The CQ bit was checked by tommi@chromium.org

[tommi (sloooow) - chröme, 2015-12-01 16:55:01]

The patchset sent to the CQ was uploaded after l-g-t-m from tommi@chromium.org
Link to the patchset: https://codereview.chromium.org/1487733003/#ps20001
(title: "Let there be just one quit message")

[commit-bot: I haz the power, 2015-12-01 16:56:05]

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

[commit-bot: I haz the power, 2015-12-01 17:10:30]

Description was changed from

==========
Allow multiple OnMoreData() calls in WASAPIAudioOutputStreamTest.ValidPacketSize

Follow up on https://codereview.chromium.org/1318933003/ and allow
OnMoreData() to be called more than once in the test.

BUG=524947
TEST=WASAPIAudioOutputStreamTest.ValidPacketSize passes
==========

to

==========
Allow multiple OnMoreData() calls in WASAPIAudioOutputStreamTest.ValidPacketSize

Follow up on https://codereview.chromium.org/1318933003/ and allow
OnMoreData() to be called more than once in the test.

BUG=524947
TEST=WASAPIAudioOutputStreamTest.ValidPacketSize passes
==========

[commit-bot: I haz the power, 2015-12-01 17:10:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-01 17:11:14]

Description was changed from

==========
Allow multiple OnMoreData() calls in WASAPIAudioOutputStreamTest.ValidPacketSize

Follow up on https://codereview.chromium.org/1318933003/ and allow
OnMoreData() to be called more than once in the test.

BUG=524947
TEST=WASAPIAudioOutputStreamTest.ValidPacketSize passes
==========

to

==========
Allow multiple OnMoreData() calls in WASAPIAudioOutputStreamTest.ValidPacketSize

Follow up on https://codereview.chromium.org/1318933003/ and allow
OnMoreData() to be called more than once in the test.

BUG=524947
TEST=WASAPIAudioOutputStreamTest.ValidPacketSize passes

Committed: https://crrev.com/7a7643ee8652ddab8503c57fe0ff1af174ea235b
Cr-Commit-Position: refs/heads/master@{#362439}
==========

[commit-bot: I haz the power, 2015-12-01 17:11:14]

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