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

Unified Diff: content/renderer/media/webrtc_audio_renderer.h

Issue 139303016: Feed the render data to MediaStreamAudioProcessor and used AudioBus in render callback (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixed the unittests, ready for review now. Created 6 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/webrtc_audio_renderer.h
diff --git a/content/renderer/media/webrtc_audio_renderer.h b/content/renderer/media/webrtc_audio_renderer.h
index bdaf1fd9f3b3004e4ddd78bec369ca4e64cb0fea..77c1d3d6f3d1f773ad787e3bf87674dc2b4488a1 100644
--- a/content/renderer/media/webrtc_audio_renderer.h
+++ b/content/renderer/media/webrtc_audio_renderer.h
@@ -52,6 +52,10 @@ class CONTENT_EXPORT WebRtcAudioRenderer
// Used to DCHECK on the expected state.
bool IsStarted() const;
+ // Accessors to the sink audio parameters.
+ int channels() const { return number_of_channels_; }
+ int sample_rate() const { return sample_rate_; }
+
private:
// MediaStreamAudioRenderer implementation. This is private since we want
// callers to use proxy objects.
@@ -100,10 +104,6 @@ class CONTENT_EXPORT WebRtcAudioRenderer
// Audio data source from the browser process.
WebRtcAudioRendererSource* source_;
- // Buffers used for temporary storage during render callbacks.
- // Allocated during initialization.
- scoped_ptr<int16[]> buffer_;
-
// Protects access to |state_|, |source_| and |sink_|.
base::Lock lock_;
@@ -124,9 +124,11 @@ class CONTENT_EXPORT WebRtcAudioRenderer
// Delay due to the FIFO in milliseconds.
int fifo_delay_milliseconds_;
- // The preferred sample rate and buffer sizes provided via the ctor.
- const int sample_rate_;
- const int frames_per_buffer_;
+ // The sample rate, number of channels and buffer sizes used by the sink of
+ // the renderer.
+ int sample_rate_;
+ int number_of_channels_;
+ int frames_per_buffer_;
DISALLOW_IMPLICIT_CONSTRUCTORS(WebRtcAudioRenderer);
};

Powered by Google App Engine
This is Rietveld 408576698