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

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

Issue 9121062: Remove "high"-latency audio code path (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase ToT yet again Created 8 years, 11 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/media/audio_message_filter.cc
diff --git a/content/renderer/media/audio_message_filter.cc b/content/renderer/media/audio_message_filter.cc
index 93e1529d80a9fb990df3d6181b44a8e900fe148f..41e236229153c21aadf523986296903b234d25fd 100644
--- a/content/renderer/media/audio_message_filter.cc
+++ b/content/renderer/media/audio_message_filter.cc
@@ -42,12 +42,8 @@ bool AudioMessageFilter::Send(IPC::Message* message) {
bool AudioMessageFilter::OnMessageReceived(const IPC::Message& message) {
bool handled = true;
IPC_BEGIN_MESSAGE_MAP(AudioMessageFilter, message)
- IPC_MESSAGE_HANDLER(AudioMsg_RequestPacket, OnRequestPacket)
IPC_MESSAGE_HANDLER(AudioMsg_NotifyStreamCreated, OnStreamCreated)
- IPC_MESSAGE_HANDLER(AudioMsg_NotifyLowLatencyStreamCreated,
- OnLowLatencyStreamCreated)
IPC_MESSAGE_HANDLER(AudioMsg_NotifyStreamStateChanged, OnStreamStateChanged)
- IPC_MESSAGE_HANDLER(AudioMsg_NotifyStreamVolume, OnStreamVolume)
IPC_MESSAGE_UNHANDLED(handled = false)
IPC_END_MESSAGE_MAP()
return handled;
@@ -67,31 +63,7 @@ void AudioMessageFilter::OnChannelClosing() {
channel_ = NULL;
}
-void AudioMessageFilter::OnRequestPacket(int stream_id,
- AudioBuffersState buffers_state) {
- Delegate* delegate = delegates_.Lookup(stream_id);
- if (!delegate) {
- DLOG(WARNING) << "Got audio packet request for a non-existent or removed"
- " audio renderer.";
- return;
- }
-
- delegate->OnRequestPacket(buffers_state);
-}
-
-void AudioMessageFilter::OnStreamCreated(int stream_id,
- base::SharedMemoryHandle handle,
- uint32 length) {
- Delegate* delegate = delegates_.Lookup(stream_id);
- if (!delegate) {
- DLOG(WARNING) << "Got audio stream event for a non-existent or removed"
- " audio renderer.";
- return;
- }
- delegate->OnCreated(handle, length);
-}
-
-void AudioMessageFilter::OnLowLatencyStreamCreated(
+void AudioMessageFilter::OnStreamCreated(
int stream_id,
base::SharedMemoryHandle handle,
#if defined(OS_WIN)
@@ -111,7 +83,7 @@ void AudioMessageFilter::OnLowLatencyStreamCreated(
base::SyncSocket socket(socket_handle);
return;
}
- delegate->OnLowLatencyCreated(handle, socket_handle, length);
+ delegate->OnStreamCreated(handle, socket_handle, length);
}
void AudioMessageFilter::OnStreamStateChanged(
@@ -125,16 +97,6 @@ void AudioMessageFilter::OnStreamStateChanged(
delegate->OnStateChanged(state);
}
-void AudioMessageFilter::OnStreamVolume(int stream_id, double volume) {
- Delegate* delegate = delegates_.Lookup(stream_id);
- if (!delegate) {
- DLOG(WARNING) << "Got audio stream event for a non-existent or removed"
- " audio renderer.";
- return;
- }
- delegate->OnVolume(volume);
-}
-
int32 AudioMessageFilter::AddDelegate(Delegate* delegate) {
return delegates_.Add(delegate);
}
« no previous file with comments | « content/renderer/media/audio_message_filter.h ('k') | content/renderer/media/audio_message_filter_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698