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

Unified Diff: content/renderer/media/peer_connection_handler_base.h

Issue 10383151: Refactor MediaStreamImpl and PeerConnection glue implementation after WebKit changes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase and added NON_EXPORTED_BASE to the base of MediaStreamExtradata. Created 8 years, 7 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
Index: content/renderer/media/peer_connection_handler_base.h
diff --git a/content/renderer/media/peer_connection_handler_base.h b/content/renderer/media/peer_connection_handler_base.h
index 482817a4cb3cfc0982752da551392c804c0bb0b0..fbb7a6918ddbcbea846dff4f0ddcd8aa0cf866ba 100644
--- a/content/renderer/media/peer_connection_handler_base.h
+++ b/content/renderer/media/peer_connection_handler_base.h
@@ -16,7 +16,6 @@
#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebMediaStreamDescriptor.h"
class MediaStreamDependencyFactory;
-class MediaStreamImpl;
// PeerConnectionHandlerBase is the base class of a delegate for the
// PeerConnection (JSEP or ROAP) API messages going between WebKit and native
@@ -25,15 +24,8 @@ class CONTENT_EXPORT PeerConnectionHandlerBase
: NON_EXPORTED_BASE(public webrtc::PeerConnectionObserver) {
public:
PeerConnectionHandlerBase(
- MediaStreamImpl* msi,
MediaStreamDependencyFactory* dependency_factory);
- webrtc::MediaStreamInterface* GetRemoteMediaStream(
- const WebKit::WebMediaStreamDescriptor& stream);
-
- void SetRemoteVideoRenderer(const std::string& source_id,
- webrtc::VideoRendererWrapperInterface* renderer);
-
protected:
virtual ~PeerConnectionHandlerBase();
@@ -41,13 +33,6 @@ class CONTENT_EXPORT PeerConnectionHandlerBase
void RemoveStream(const WebKit::WebMediaStreamDescriptor& stream);
WebKit::WebMediaStreamDescriptor CreateWebKitStreamDescriptor(
webrtc::MediaStreamInterface* stream);
- // Returns a VideoTrack given the |source_id|
- webrtc::VideoTrackInterface* FindRemoteVideoTrack(
- const std::string& source_id);
-
- // media_stream_impl_ is a raw pointer, and is valid for the lifetime of this
- // class. Calls to it must be done on the render thread.
- MediaStreamImpl* media_stream_impl_;
// dependency_factory_ is a raw pointer, and is valid for the lifetime of
// MediaStreamImpl.
@@ -62,15 +47,6 @@ class CONTENT_EXPORT PeerConnectionHandlerBase
WebKit::WebMediaStreamDescriptor> RemoteStreamMap;
RemoteStreamMap remote_streams_;
- // Native PeerConnection only supports 1:1 mapping between MediaStream and
- // video tag/renderer, so we restrict to this too. The key in
- // VideoRendererMap is the track label.
- typedef talk_base::scoped_refptr<webrtc::VideoRendererWrapperInterface>
- VideoRendererPtr;
-
- typedef std::map<std::string, VideoRendererPtr> VideoRendererMap;
- VideoRendererMap video_renderers_;
-
// The message loop we are created on and on which to make calls to WebKit.
// This should be the render thread message loop.
scoped_refptr<base::MessageLoopProxy> message_loop_proxy_;
« no previous file with comments | « content/renderer/media/peer_connection_handler.cc ('k') | content/renderer/media/peer_connection_handler_base.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698