Fix flaky test WebRtcMediaRecorderTest.PeerConnection

Fix flaky test WebRtcMediaRecorderTest.PeerConnection

A previous CL changed from logging to DCHECKing when setting minimum playout delay on a VoE channel: https://codereview.webrtc.org/2452163004/

I thought it safe at the time, since the input parameter range is capped, but apparently I didn't dig deep enough, as ultimately a failure may be returned for other reasons: https://chromium.googlesource.com/external/webrtc/+/master/webrtc/modules/audio_coding/neteq/delay_manager.cc#381

Thus, reverting to old behavior.

BUG=694373
Review-Url: https://codereview.webrtc.org/2704933008
Cr-Commit-Position: refs/heads/master@{#16775}
Committed: https://chromium.googlesource.com/external/webrtc/+/0335e6c4bfa9a3254a392c003344c71238ee832f

[the sun, 2017-02-22 10:26:43]

solenberg@webrtc.org changed reviewers:
+ henrik.lundin@webrtc.org

[hlundin-webrtc, 2017-02-22 13:10:05]

lgtm

[the sun, 2017-02-22 13:44:24]

The CQ bit was checked by solenberg@webrtc.org

[commit-bot: I haz the power, 2017-02-22 13:44:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-02-22 15:07:06]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487771064808740,
"parent_rev": "1d4e3d8a2ec0a138d1340e0d8643d69313a2194b", "commit_rev":
"0335e6c4bfa9a3254a392c003344c71238ee832f"}

[commit-bot: I haz the power, 2017-02-22 15:07:09]

Description was changed from

==========
Fix flaky test WebRtcMediaRecorderTest.PeerConnection

A previous CL changed from logging to DCHECKing when setting minimum playout
delay on a VoE channel: https://codereview.webrtc.org/2452163004/

I thought it safe at the time, since the input parameter range is capped, but
apparently I didn't dig deep enough, as ultimately a failure may be returned for
other reasons:
https://chromium.googlesource.com/external/webrtc/+/master/webrtc/modules/aud...

Thus, reverting to old behavior.

BUG=694373
==========

to

==========
Fix flaky test WebRtcMediaRecorderTest.PeerConnection

A previous CL changed from logging to DCHECKing when setting minimum playout
delay on a VoE channel: https://codereview.webrtc.org/2452163004/

I thought it safe at the time, since the input parameter range is capped, but
apparently I didn't dig deep enough, as ultimately a failure may be returned for
other reasons:
https://chromium.googlesource.com/external/webrtc/+/master/webrtc/modules/aud...

Thus, reverting to old behavior.

BUG=694373

Review-Url: https://codereview.webrtc.org/2704933008
Cr-Commit-Position: refs/heads/master@{#16775}
Committed:
https://chromium.googlesource.com/external/webrtc/+/0335e6c4bfa9a3254a392c003...
==========

[commit-bot: I haz the power, 2017-02-22 15:07:11]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/0335e6c4bfa9a3254a392c003...