Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Unified Diff: trunk/src/content/renderer/render_thread_impl.cc

Issue 135683002: Revert 244074 "Eliminate video capture thread in renderer" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « trunk/src/content/renderer/render_thread_impl.h ('k') | trunk/src/media/media.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/content/renderer/render_thread_impl.cc
===================================================================
--- trunk/src/content/renderer/render_thread_impl.cc (revision 244357)
+++ trunk/src/content/renderer/render_thread_impl.cc (working copy)
@@ -347,7 +347,7 @@
db_message_filter_ = new DBMessageFilter();
AddFilter(db_message_filter_.get());
- vc_manager_.reset(new VideoCaptureImplManager());
+ vc_manager_ = new VideoCaptureImplManager();
AddFilter(vc_manager_->video_capture_message_filter());
#if defined(ENABLE_WEBRTC)
@@ -361,7 +361,7 @@
webrtc_identity_service_.reset(new WebRTCIdentityService());
media_stream_factory_.reset(new MediaStreamDependencyFactory(
- p2p_socket_dispatcher_.get()));
+ vc_manager_.get(), p2p_socket_dispatcher_.get()));
AddObserver(media_stream_factory_.get());
#endif // defined(ENABLE_WEBRTC)
@@ -464,13 +464,7 @@
RemoveFilter(audio_message_filter_.get());
audio_message_filter_ = NULL;
- // |media_stream_factory_| produces users of |vc_manager_| so it must be
- // destroyed first.
-#if defined(ENABLE_WEBRTC)
- media_stream_factory_.reset();
-#endif
RemoveFilter(vc_manager_->video_capture_message_filter());
- vc_manager_.reset();
RemoveFilter(db_message_filter_.get());
db_message_filter_ = NULL;
« no previous file with comments | « trunk/src/content/renderer/render_thread_impl.h ('k') | trunk/src/media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698