Wrote a test which exercises audio-only WebRTC calls.

Wrote a test which exercises audio-only WebRTC calls.

Depends on https://codereview.chromium.org/260073013/ since it requires audio beeps generated even when the fake video device hasn't been acquired.

BUG=None

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275790

[phoglund_chromium, 2014-04-29 15:58:19]

Nice Shijing, your patch made the test run perfectly! Please review this one
after you have landed your patch.

[no longer working on chromium, 2014-06-05 09:12:32]

lgtm with one nit.

https://codereview.chromium.org/252703003/diff/20001/content/test/data/media/...
File content/test/data/media/webrtc_test_audio.js (right):

https://codereview.chromium.org/252703003/diff/20001/content/test/data/media/...
content/test/data/media/webrtc_test_audio.js:21: return;  // The call probably
isn't up yet.
nit, make it look more chromium:
if (audioOutputLevels.length) {
  // The call probably isn't up yet.
  return;
}

[phoglund_chromium, 2014-06-09 06:55:36]

https://codereview.chromium.org/252703003/diff/20001/content/test/data/media/...
File content/test/data/media/webrtc_test_audio.js (right):

https://codereview.chromium.org/252703003/diff/20001/content/test/data/media/...
content/test/data/media/webrtc_test_audio.js:21: return;  // The call probably
isn't up yet.
On 2014/06/05 09:12:32, xians1 wrote:
> nit, make it look more chromium:
> if (audioOutputLevels.length) {
>   // The call probably isn't up yet.
>   return;
> }

Done.

[phoglund_chromium, 2014-06-09 06:55:40]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2014-06-09 06:55:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/phoglund@chromium.org/252703003/40001

[commit-bot: I haz the power, 2014-06-09 11:20:27]

Change committed as 275790