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

Unified Diff: Source/modules/mediastream/MediaStream.h

Issue 16753003: Revert "Remove MediaStreamDescriptor and call/use WebMediaStream directly" (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 6 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: Source/modules/mediastream/MediaStream.h
diff --git a/Source/modules/mediastream/MediaStream.h b/Source/modules/mediastream/MediaStream.h
index f066155dc210e143ad396804ca6e7079e495d7ad..44926c4c54f86ce25ff679b867ecb488bf5b33c6 100644
--- a/Source/modules/mediastream/MediaStream.h
+++ b/Source/modules/mediastream/MediaStream.h
@@ -32,26 +32,25 @@
#include "core/dom/ExceptionBase.h"
#include "core/html/URLRegistry.h"
#include "core/platform/Timer.h"
-#include "core/platform/chromium/support/WebMediaStreamClient.h"
+#include "core/platform/mediastream/MediaStreamDescriptor.h"
#include "modules/mediastream/MediaStreamTrack.h"
-#include "public/platform/WebMediaStream.h"
#include "wtf/RefCounted.h"
#include "wtf/RefPtr.h"
namespace WebCore {
-class MediaStream : public RefCounted<MediaStream>, public ScriptWrappable, public URLRegistrable, public WebKit::WebMediaStreamClient, public EventTarget, public ContextDestructionObserver {
+class MediaStream : public RefCounted<MediaStream>, public ScriptWrappable, public URLRegistrable, public MediaStreamDescriptorClient, public EventTarget, public ContextDestructionObserver {
public:
static PassRefPtr<MediaStream> create(ScriptExecutionContext*);
static PassRefPtr<MediaStream> create(ScriptExecutionContext*, PassRefPtr<MediaStream>);
static PassRefPtr<MediaStream> create(ScriptExecutionContext*, const MediaStreamTrackVector&);
- static PassRefPtr<MediaStream> create(ScriptExecutionContext*, WebKit::WebMediaStream);
+ static PassRefPtr<MediaStream> create(ScriptExecutionContext*, PassRefPtr<MediaStreamDescriptor>);
virtual ~MediaStream();
// DEPRECATED
- String label() const { return m_webStream.id(); }
+ String label() const { return m_descriptor->id(); }
- String id() const { return m_webStream.id(); }
+ String id() const { return m_descriptor->id(); }
void addTrack(PassRefPtr<MediaStreamTrack>, ExceptionCode&);
void removeTrack(PassRefPtr<MediaStreamTrack>, ExceptionCode&);
@@ -67,11 +66,11 @@ public:
DEFINE_ATTRIBUTE_EVENT_LISTENER(addtrack);
DEFINE_ATTRIBUTE_EVENT_LISTENER(removetrack);
- // WebMediaStreamClient
+ // MediaStreamDescriptorClient
virtual void trackEnded() OVERRIDE;
virtual void streamEnded() OVERRIDE;
- WebKit::WebMediaStream webStream() const { return m_webStream; }
+ MediaStreamDescriptor* descriptor() const { return m_descriptor.get(); }
// EventTarget
virtual const AtomicString& interfaceName() const OVERRIDE;
@@ -84,7 +83,7 @@ public:
virtual URLRegistry& registry() const OVERRIDE;
protected:
- MediaStream(ScriptExecutionContext*, WebKit::WebMediaStream);
+ MediaStream(ScriptExecutionContext*, PassRefPtr<MediaStreamDescriptor>);
// EventTarget
virtual EventTargetData* eventTargetData() OVERRIDE;
@@ -98,7 +97,7 @@ private:
virtual void refEventTarget() OVERRIDE { ref(); }
virtual void derefEventTarget() OVERRIDE { deref(); }
- // WebMediaStreamClient
+ // MediaStreamDescriptorClient
virtual void addRemoteTrack(MediaStreamComponent*) OVERRIDE;
virtual void removeRemoteTrack(MediaStreamComponent*) OVERRIDE;
@@ -111,7 +110,7 @@ private:
MediaStreamTrackVector m_audioTracks;
MediaStreamTrackVector m_videoTracks;
- WebKit::WebMediaStream m_webStream;
+ RefPtr<MediaStreamDescriptor> m_descriptor;
Timer<MediaStream> m_scheduledEventTimer;
Vector<RefPtr<Event> > m_scheduledEvents;

Powered by Google App Engine
This is Rietveld 408576698