Don't recreate the speech encoder if we don't have to

webrtc/modules/audio_coding/codecs/audio_encoder.h

Index: webrtc/modules/audio_coding/codecs/audio_encoder.h
diff --git a/webrtc/modules/audio_coding/codecs/audio_encoder.h b/webrtc/modules/audio_coding/codecs/audio_encoder.h
index f4c5794d92d5d902d6df688317e0db16cb3e9ba4..ecc28d96a16239de05e39f04b3f41676f2d3f380 100644
--- a/webrtc/modules/audio_coding/codecs/audio_encoder.h
+++ b/webrtc/modules/audio_coding/codecs/audio_encoder.h
@@ -149,6 +149,15 @@ class AudioEncoder {
   // implementation does the latter).
   virtual void SetTargetBitrate(int target_bps);
 
+  // Causes this encoder to let go of any other encoders it contains, and
+  // returns a pointer to an array where they are stored (which is required to
+  // live as long as this encoder). Unless the returned array is empty, you may
+  // not call any methods on this encoder afterwards, except for the
+  // destructor. The default implementation just returns an empty array.
+  // NOTE: This method is subject to change. Do not call or override it.
+  virtual rtc::ArrayView<std::unique_ptr<AudioEncoder>>
+  ReclaimContainedEncoders();

[kwiberg-webrtc, 2016/06/22 13:46:12]

This method is something of a hack. Hopefully, we'll be able to get rid of it
when we implement the prophesied encoder factory (since we should then set all
encoder properties at creation, and thus will lose no config state by recreating
encoders, which we should not end up doing anyway because we don't set a bunch
of properties after the fact).

+
  protected:
   // Subclasses implement this to perform the actual encoding. Called by
   // Encode().