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

Unified Diff: content/browser/renderer_host/media/audio_renderer_host.h

Issue 12383016: Merge AssociateStreamWithProducer message into CreateStream message for both audio output and input. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 8 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.h
diff --git a/content/browser/renderer_host/media/audio_renderer_host.h b/content/browser/renderer_host/media/audio_renderer_host.h
index 917dc2aa0f5d6e9148118a81b3d7a8aa94c0e56c..803539cf48a1c7bdd331ff28e27fd03338ce0b18 100644
--- a/content/browser/renderer_host/media/audio_renderer_host.h
+++ b/content/browser/renderer_host/media/audio_renderer_host.h
@@ -93,15 +93,14 @@ class CONTENT_EXPORT AudioRendererHost : public BrowserMessageFilter {
// Methods called on IO thread ----------------------------------------------
// Audio related IPC message handlers.
- // Creates an audio output stream with the specified format. If this call is
- // successful this object would keep an internal entry of the stream for the
- // required properties.
- void OnCreateStream(int stream_id,
- const media::AudioParameters& params);
- // Track that the data for the audio stream referenced by |stream_id| is
+ // Creates an audio output stream with the specified format whose data is
// produced by an entity in the render view referenced by |render_view_id|.
- void OnAssociateStreamWithProducer(int stream_id, int render_view_id);
+ // Upon success/failure, the peer is notified via the NotifyStreamCreated
+ // message.
+ void OnCreateStream(int stream_id,
+ int render_view_id,
+ const media::AudioParameters& params);
// Play the audio stream referenced by |stream_id|.
void OnPlayStream(int stream_id);
@@ -116,7 +115,8 @@ class CONTENT_EXPORT AudioRendererHost : public BrowserMessageFilter {
void OnSetVolume(int stream_id, double volume);
// Complete the process of creating an audio stream. This will set up the
- // shared memory or shared socket in low latency mode.
+ // shared memory or shared socket in low latency mode and send the
+ // NotifyStreamCreated message to the peer.
void DoCompleteCreation(AudioEntry* entry);
// Propagate audible signal to MediaObserver.

Powered by Google App Engine
This is Rietveld 408576698