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

Unified Diff: media/base/audio_renderer_mixer_pool.h

Issue 1942803002: Caching AudioOutputDevice instances in mixer manager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Make WebAudioSourceProvider to always return real sink info reguardless the client - to avoid behavior change. Created 4 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_renderer_mixer_input_unittest.cc ('k') | media/base/audio_renderer_mixer_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/audio_renderer_mixer_pool.h
diff --git a/media/base/audio_renderer_mixer_pool.h b/media/base/audio_renderer_mixer_pool.h
new file mode 100644
index 0000000000000000000000000000000000000000..e6ee9f006ed6b2f1f1bf8d5d99195a198c56a863
--- /dev/null
+++ b/media/base/audio_renderer_mixer_pool.h
@@ -0,0 +1,56 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef MEDIA_BASE_AUDIO_RENDERER_MIXER_POOL_H_
+#define MEDIA_BASE_AUDIO_RENDERER_MIXER_POOL_H_
+
+#include <string>
+
+#include "media/base/output_device_info.h"
+
+namespace url {
+class Origin;
+}
+
+namespace media {
+class AudioParameters;
+class AudioRendererMixer;
+
+// Provides AudioRendererMixer instances for shared usage.
+// Thread safe.
+class MEDIA_EXPORT AudioRendererMixerPool {
+ public:
+ AudioRendererMixerPool() {}
+ virtual ~AudioRendererMixerPool() {}
+
+ // Obtains a pointer to mixer instance based on AudioParameters. The pointer
+ // is guaranteed to be valid (at least) until it's rereleased by a call to
+ // ReturnMixer().
+ virtual AudioRendererMixer* GetMixer(int owner_id,
+ const AudioParameters& params,
+ const std::string& device_id,
+ const url::Origin& security_origin,
+ OutputDeviceStatus* device_status) = 0;
+
+ // Returns mixer back to the pool, must be called when the mixer is not needed
+ // any more to avoid memory leakage.
+ virtual void ReturnMixer(int owner_id,
+ const AudioParameters& params,
+ const std::string& device_id,
+ const url::Origin& security_origin) = 0;
+
+ // Returns output device information
+ virtual OutputDeviceInfo GetOutputDeviceInfo(
+ int owner_id,
+ int session_id,
+ const std::string& device_id,
+ const url::Origin& security_origin) = 0;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(AudioRendererMixerPool);
+};
+
+} // namespace media
+
+#endif // MEDIA_BASE_AUDIO_RENDERER_MIXER_POOL_H_
« no previous file with comments | « media/base/audio_renderer_mixer_input_unittest.cc ('k') | media/base/audio_renderer_mixer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698