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

Unified Diff: content/renderer/render_thread_impl.cc

Issue 2120273004: Getting rid of AudioHardwareConfig and its synchronous IPC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased to guidou@ changes, dropped unknown frame id and default param caching Created 4 years, 5 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/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 660b52bedbb249e4e9c8877f6dd009aa38083e2b..d4cb262150b20a9471ee5ca52c736e78ab1bf4ca 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -133,7 +133,6 @@
#include "ipc/ipc_channel_handle.h"
#include "ipc/ipc_channel_mojo.h"
#include "ipc/ipc_platform_file.h"
-#include "media/base/audio_hardware_config.h"
#include "media/base/media.h"
#include "media/renderers/gpu_video_accelerator_factories.h"
#include "mojo/common/common_type_converters.h"
@@ -1539,20 +1538,6 @@ AudioRendererMixerManager* RenderThreadImpl::GetAudioRendererMixerManager() {
return audio_renderer_mixer_manager_.get();
}
-media::AudioHardwareConfig* RenderThreadImpl::GetAudioHardwareConfig() {
- if (!audio_hardware_config_) {
- media::AudioParameters input_params;
- media::AudioParameters output_params;
- Send(new ViewHostMsg_GetAudioHardwareConfig(
- &input_params, &output_params));
-
- audio_hardware_config_.reset(new media::AudioHardwareConfig(
- input_params, output_params));
- }
-
- return audio_hardware_config_.get();
-}
-
base::WaitableEvent* RenderThreadImpl::GetShutdownEvent() {
return ChildProcess::current()->GetShutDownEvent();
}

Powered by Google App Engine
This is Rietveld 408576698