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

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

Issue 9121062: Remove "high"-latency audio code path (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more to delete! 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/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 16e2d02120c78840612ffbb016f9a20f4c707618..0b1154dcf054fff4c1554df82adcf599bc8d2310 100644
--- a/content/browser/renderer_host/media/audio_renderer_host.h
+++ b/content/browser/renderer_host/media/audio_renderer_host.h
@@ -97,8 +97,6 @@ class CONTENT_EXPORT AudioRendererHost
// ownership of the reader.
scoped_ptr<media::AudioOutputController::SyncReader> reader;
- bool pending_buffer_request;
-
// Set to true after we called Close() for the controller.
bool pending_close;
};
@@ -120,8 +118,6 @@ class CONTENT_EXPORT AudioRendererHost
virtual void OnPaused(media::AudioOutputController* controller) OVERRIDE;
virtual void OnError(media::AudioOutputController* controller,
int error_code) OVERRIDE;
- virtual void OnMoreData(media::AudioOutputController* controller,
- AudioBuffersState buffers_state) OVERRIDE;
private:
friend class AudioRendererHostTest;
@@ -139,9 +135,7 @@ class CONTENT_EXPORT AudioRendererHost
// 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 AudioParameters& params,
- bool low_latency);
+ void OnCreateStream(int stream_id, const AudioParameters& params);
// Play the audio stream referenced by |stream_id|.
void OnPlayStream(int stream_id);
@@ -149,7 +143,7 @@ class CONTENT_EXPORT AudioRendererHost
// Pause the audio stream referenced by |stream_id|.
void OnPauseStream(int stream_id);
- // Discard all audio data in stream referenced by |stream_id|.
+ // Discard all audio data in stream referenced by |stream_id|.
void OnFlushStream(int stream_id);
// Close the audio stream referenced by |stream_id|.
@@ -158,12 +152,6 @@ class CONTENT_EXPORT AudioRendererHost
// Set the volume of the audio stream referenced by |stream_id|.
void OnSetVolume(int stream_id, double volume);
henrika (OOO until Aug 14) 2012/01/29 20:54:00 Why has OnGetVolume() been removed?
vrk (LEFT CHROMIUM) 2012/01/31 23:53:08 As I was deleting unused code, I noticed that OnGe
- // Get the volume of the audio stream referenced by |stream_id|.
- void OnGetVolume(int stream_id);
-
- // Notify packet has been prepared for the audio stream.
- void OnNotifyPacketReady(int stream_id, uint32 packet_size);
-
// Complete the process of creating an audio stream. This will set up the
// shared memory or shared socket in low latency mode.
void DoCompleteCreation(media::AudioOutputController* controller);
@@ -172,11 +160,6 @@ class CONTENT_EXPORT AudioRendererHost
void DoSendPlayingMessage(media::AudioOutputController* controller);
void DoSendPausedMessage(media::AudioOutputController* controller);
- // Request more data from the renderer. This method is used only in normal
- // latency mode.
- void DoRequestMoreData(media::AudioOutputController* controller,
- AudioBuffersState buffers_state);
-
// Handle error coming from audio stream.
void DoHandleError(media::AudioOutputController* controller, int error_code);

Powered by Google App Engine
This is Rietveld 408576698