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

Unified Diff: content/renderer/render_thread_impl.cc

Issue 2890753003: Introduce AudioIPCFactory. (Closed)
Patch Set: Dale's comments. Created 3 years, 7 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
« no previous file with comments | « content/renderer/render_thread_impl.h ('k') | content/test/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 875bb9d122c57b11c778ee54051e32798544e72e..8ab07806770bd83056d3779551a8819a1650b3a4 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -725,8 +725,13 @@ void RenderThreadImpl::Init(
audio_input_message_filter_ = new AudioInputMessageFilter(GetIOTaskRunner());
AddFilter(audio_input_message_filter_.get());
- audio_message_filter_ = new AudioMessageFilter(GetIOTaskRunner());
- AddFilter(audio_message_filter_.get());
+ auto audio_message_filter =
+ base::MakeRefCounted<AudioMessageFilter>(GetIOTaskRunner());
+ AddFilter(audio_message_filter.get());
+ // TODO(maxmorin): Based on a feature flag, don't create the
+ // AudioMessageFilter, making AudioIPCFactory instead use mojo factories.
+ audio_ipc_factory_.emplace(std::move(audio_message_filter),
+ GetIOTaskRunner());
midi_message_filter_ = new MidiMessageFilter(GetIOTaskRunner());
AddFilter(midi_message_filter_.get());
« no previous file with comments | « content/renderer/render_thread_impl.h ('k') | content/test/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698