Mixing audio with different latency requirements

media/base/audio_renderer_mixer_pool.h

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
index e6ee9f006ed6b2f1f1bf8d5d99195a198c56a863..d7699faf3d3c480d648963d655c7b0b9b1cc3b49 100644
--- a/media/base/audio_renderer_mixer_pool.h
+++ b/media/base/audio_renderer_mixer_pool.h
@@ -7,6 +7,7 @@
 
 #include <string>
 
+#include "media/base/audio_latency.h"
 #include "media/base/output_device_info.h"
 
 namespace url {
@@ -29,16 +30,14 @@ class MEDIA_EXPORT AudioRendererMixerPool {
   // ReturnMixer().
   virtual AudioRendererMixer* GetMixer(int owner_id,
                                        const AudioParameters& params,
+                                       AudioLatency::LatencyType latency,
                                        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;
+  virtual void ReturnMixer(const AudioRendererMixer* mixer) = 0;

[tommi (sloooow) - chröme, 2016/06/28 14:14:54]

why const?

I'm thinking that the "Get" method returns a non-const pointer, this returns a
mixer back, which will make it "writable" again, then having the const qualifier
might be misleading (assuming this pointer might be returned from GetMixer at
some point in the future)

[o1ka, 2016/06/29 10:11:26]

Done.

 
   // Returns output device information
   virtual OutputDeviceInfo GetOutputDeviceInfo(