Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1781)

Unified Diff: content/renderer/media/webrtc_audio_device_unittest.cc

Issue 11359196: Associate audio streams with their source/destination RenderView. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Restored AudioDeviceFactory. Created new RendererAudioOutputDevice. Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/media/webrtc_audio_device_unittest.cc
diff --git a/content/renderer/media/webrtc_audio_device_unittest.cc b/content/renderer/media/webrtc_audio_device_unittest.cc
index ee160c9dbf3748ecbbb4da716613e556a0e3a744..a1730feb946db8e81a79f2b288c6f6f72fe98bb3 100644
--- a/content/renderer/media/webrtc_audio_device_unittest.cc
+++ b/content/renderer/media/webrtc_audio_device_unittest.cc
@@ -27,6 +27,8 @@ namespace content {
namespace {
+const int kRenderViewId = 1;
+
class AudioUtil : public AudioUtilInterface {
public:
AudioUtil() {}
@@ -265,7 +267,8 @@ TEST_F(WebRTCAudioDeviceTest, StartPlayout) {
EXPECT_CALL(media_observer(),
OnDeleteAudioStream(_, 1)).Times(AnyNumber());
- scoped_refptr<WebRtcAudioRenderer> renderer = new WebRtcAudioRenderer();
+ scoped_refptr<WebRtcAudioRenderer> renderer =
+ new WebRtcAudioRenderer(kRenderViewId);
scoped_refptr<WebRtcAudioDeviceImpl> webrtc_audio_device(
new WebRtcAudioDeviceImpl());
webrtc_audio_device->SetSessionId(1);
@@ -414,8 +417,8 @@ TEST_F(WebRTCAudioDeviceTest, PlayLocalFile) {
EXPECT_CALL(media_observer(),
OnDeleteAudioStream(_, 1)).Times(AnyNumber());
- scoped_refptr<WebRtcAudioRenderer> renderer = new WebRtcAudioRenderer();
-
+ scoped_refptr<WebRtcAudioRenderer> renderer =
+ new WebRtcAudioRenderer(kRenderViewId);
scoped_refptr<WebRtcAudioDeviceImpl> webrtc_audio_device(
new WebRtcAudioDeviceImpl());
webrtc_audio_device->SetSessionId(1);
@@ -486,8 +489,8 @@ TEST_F(WebRTCAudioDeviceTest, FullDuplexAudioWithAGC) {
EXPECT_CALL(media_observer(),
OnDeleteAudioStream(_, 1)).Times(AnyNumber());
- scoped_refptr<WebRtcAudioRenderer> renderer = new WebRtcAudioRenderer();
-
+ scoped_refptr<WebRtcAudioRenderer> renderer =
+ new WebRtcAudioRenderer(kRenderViewId);
scoped_refptr<WebRtcAudioDeviceImpl> webrtc_audio_device(
new WebRtcAudioDeviceImpl());
webrtc_audio_device->SetSessionId(1);

Powered by Google App Engine
This is Rietveld 408576698