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

Unified Diff: content/browser/renderer_host/media/audio_renderer_host_unittest.cc

Issue 324143002: Decouple IPC::MessageFilter from IPC::Channel (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixing compilation errors Created 6 years, 6 months 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/browser/renderer_host/media/audio_renderer_host_unittest.cc
diff --git a/content/browser/renderer_host/media/audio_renderer_host_unittest.cc b/content/browser/renderer_host/media/audio_renderer_host_unittest.cc
index 343c9890810ed2e6271dcc20dbb11986c1287d3c..401f183723425a14e579910ab715bed0ba351ed9 100644
--- a/content/browser/renderer_host/media/audio_renderer_host_unittest.cc
+++ b/content/browser/renderer_host/media/audio_renderer_host_unittest.cc
@@ -7,6 +7,7 @@
#include "base/memory/scoped_ptr.h"
#include "base/run_loop.h"
#include "base/sync_socket.h"
+#include "content/browser/browser_message_filter_peer.h"
#include "content/browser/media/capture/audio_mirroring_manager.h"
#include "content/browser/media/media_internals.h"
#include "content/browser/renderer_host/media/audio_input_device_manager.h"
@@ -166,13 +167,14 @@ class AudioRendererHostTest : public testing::Test {
media_stream_manager_.get());
// Simulate IPC channel connected.
- host_->set_peer_pid_for_testing(base::GetCurrentProcId());
+ host_peer_.reset(new BrowserMessageFilterPeer(base::GetCurrentProcId()));
+ host_->OnFilterAddedToHost(host_peer_.get());
}
virtual ~AudioRendererHostTest() {
// Simulate closing the IPC channel and give the audio thread time to close
// the underlying streams.
- host_->OnChannelClosing();
+ host_->OnSenderClosing();
SyncWithAudioThread();
// Release the reference to the mock object. The object will be destructed
@@ -282,6 +284,7 @@ class AudioRendererHostTest : public testing::Test {
scoped_ptr<media::AudioManager> audio_manager_;
MockAudioMirroringManager mirroring_manager_;
scoped_refptr<MockAudioRendererHost> host_;
+ scoped_ptr<BrowserMessageFilterPeer> host_peer_;
DISALLOW_COPY_AND_ASSIGN(AudioRendererHostTest);
};

Powered by Google App Engine
This is Rietveld 408576698