Revert to recreating RTCVideoEncoder::Impl

content/renderer/media/rtc_video_encoder.cc

Index: content/renderer/media/rtc_video_encoder.cc
diff --git a/content/renderer/media/rtc_video_encoder.cc b/content/renderer/media/rtc_video_encoder.cc
index 2a2891cc07fc3ee3591bd764498696fc1dcbfe70..a6f6fa42d2f74b6fec840b7f8b525b15a0c2794c 100644
--- a/content/renderer/media/rtc_video_encoder.cc
+++ b/content/renderer/media/rtc_video_encoder.cc
@@ -386,6 +386,12 @@ void RTCVideoEncoder::Impl::Destroy(base::WaitableEvent* async_waiter) {
     video_encoder_.reset();
     SetStatus(WEBRTC_VIDEO_CODEC_UNINITIALIZED);
   }
+  input_buffers_.clear();

[Pawel Osciak, 2016/04/28 02:05:33]

What would you think of having a Clear() (or a similar) method instead?

We could call it from here and the constructor. This is specifically for
picture_id_ as it's not obvious what it should be initialized to and it'd be
best to always init uniformly...

[emircan, 2016/04/28 02:44:09]

Done.

+  output_buffers_.clear();
+  input_buffers_free_.clear();
+  output_buffers_free_count_ = 0;
+  input_next_frame_keyframe_ = false;
+  picture_id_ = static_cast<uint16_t>(base::RandInt(0, 0x7FFF));
   async_waiter->Signal();
 }