AudioBus: Add a ToInterleavedFloat() method to AudioBus

media/base/audio_bus.cc

Index: media/base/audio_bus.cc
diff --git a/media/base/audio_bus.cc b/media/base/audio_bus.cc
index 8b7fa3dde87589dc5188791d58e6868778b3c901..bbc5da1f6aec39bbbd313ef5d07065f04a3774c1 100644
--- a/media/base/audio_bus.cc
+++ b/media/base/audio_bus.cc
@@ -300,6 +300,22 @@ void AudioBus::ToInterleaved(int frames, int bytes_per_sample,
   ToInterleavedPartial(0, frames, bytes_per_sample, dest);
 }
 
+// Interleaves |audio_bus| channels() of floats into a single output linear
+// |buffer|.
+void AudioBus::ToInterleavedFloat(int source_offset,
+                                  int destination_offset,
+                                  int num_samples,
+                                  float* buffer) const {
+  CheckOverflow(source_offset, destination_offset, num_samples);
+  for (int ch = 0; ch < channels(); ++ch) {
+    const float* src = channel(ch) + source_offset;
+    const float* const src_end = src + num_samples;
+    float* dest = buffer + destination_offset + ch;
+    for (; src < src_end; ++src, dest += channels())

[mcasas, 2016/06/02 01:13:14]

nit: every cycle counts, so get channels() evaluation out
of both loops:
 const int channels = channels();

[chfremer, 2016/06/02 18:13:33]

As per the discussion in the corresponding bug entry [1], I am not sure whether
or not we will keep this newly added code. Will address this if we decide to
keep it.

[1] https://bugs.chromium.org/p/chromium/issues/detail?id=580391

+      *dest = *src;
+  }
+}
+
 // TODO(dalecurtis): See if intrinsic optimizations help any here.
 void AudioBus::ToInterleavedPartial(int start_frame, int frames,
                                     int bytes_per_sample, void* dest) const {