Destroy all rtc peer connection handlers before shutting down blink

content/renderer/render_thread_impl.cc

Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index dd75520bbd51798da537b82a36a2d04de1b0bb28..546d4ef0efd493b3ff0f18e65a2f58d8943f3f33 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -79,6 +79,7 @@
 #include "content/renderer/media/midi_message_filter.h"
 #include "content/renderer/media/peer_connection_tracker.h"
 #include "content/renderer/media/renderer_gpu_video_accelerator_factories.h"
+#include "content/renderer/media/rtc_peer_connection_handler.h"
 #include "content/renderer/media/video_capture_impl_manager.h"
 #include "content/renderer/media/video_capture_message_filter.h"
 #include "content/renderer/media/webrtc/peer_connection_dependency_factory.h"
@@ -565,6 +566,8 @@ void RenderThreadImpl::Shutdown() {
   // hold pointers to V8 objects (e.g., via pending requests).
   main_thread_indexed_db_dispatcher_.reset();
 
+  RTCPeerConnectionHandler::DestructAllHandlers();

[perkj_chrome, 2014/05/22 06:45:26]

This should be done before |media_stream_factory_| is reset within the #if
defined(ENABLE_WEBRTC), see content_renderer.gypi rtc_peer_connection_handler.cc
. (And it looks like I should rename |media_stream_factory_| to
|peer_connection_factory_| here as well.  

[jochen (gone - plz use gerrit), 2014/05/22 07:53:44]

Done.

+
   if (webkit_platform_support_)
     blink::shutdown();