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

Unified Diff: media/base/audio_renderer_mixer.h

Issue 301223012: Deliver RenderCallbackErrors even when mixer inputs are paused. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: List! Created 6 years, 7 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
« no previous file with comments | « media/base/audio_converter.h ('k') | media/base/audio_renderer_mixer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/audio_renderer_mixer.h
diff --git a/media/base/audio_renderer_mixer.h b/media/base/audio_renderer_mixer.h
index 942c61fe8495cd3b0a921bf0a12dc7da10269b29..4b0af9b4e3f60034098d9fa3dcb19f597a3d9795 100644
--- a/media/base/audio_renderer_mixer.h
+++ b/media/base/audio_renderer_mixer.h
@@ -26,10 +26,15 @@ class MEDIA_EXPORT AudioRendererMixer
virtual ~AudioRendererMixer();
// Add or remove a mixer input from mixing; called by AudioRendererMixerInput.
- void AddMixerInput(AudioConverter::InputCallback* input,
- const base::Closure& error_cb);
+ void AddMixerInput(AudioConverter::InputCallback* input);
void RemoveMixerInput(AudioConverter::InputCallback* input);
+ // Since errors may occur even when no inputs are playing, an error callback
+ // must be registered separately from adding a mixer input. The same callback
+ // must be given to both the functions.
+ void AddErrorCallback(const base::Closure& error_cb);
+ void RemoveErrorCallback(const base::Closure& error_cb);
+
void set_pause_delay_for_testing(base::TimeDelta delay) {
pause_delay_ = delay;
}
@@ -43,12 +48,12 @@ class MEDIA_EXPORT AudioRendererMixer
// Output sink for this mixer.
scoped_refptr<AudioRendererSink> audio_sink_;
- // Set of mixer inputs to be mixed by this mixer. Access is thread-safe
- // through |mixer_inputs_lock_|.
- typedef std::map<AudioConverter::InputCallback*, base::Closure>
- AudioRendererMixerInputSet;
- AudioRendererMixerInputSet mixer_inputs_;
- base::Lock mixer_inputs_lock_;
+ // ---------------[ All variables below protected by |lock_| ]---------------
+ base::Lock lock_;
+
+ // List of error callbacks used by this mixer.
+ typedef std::list<base::Closure> ErrorCallbackList;
+ ErrorCallbackList error_callbacks_;
// Handles mixing and resampling between input and output parameters.
AudioConverter audio_converter_;
« no previous file with comments | « media/base/audio_converter.h ('k') | media/base/audio_renderer_mixer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698