Implement HTMLMediaElement::srcObject.

Source/modules/mediastream/MediaStream.cpp

Index: Source/modules/mediastream/MediaStream.cpp
diff --git a/Source/modules/mediastream/MediaStream.cpp b/Source/modules/mediastream/MediaStream.cpp
index 86ffd4fea04c9bc0b58388124190c96065b1f3af..6db5bfded8d522e82285f69f5181ff46c8b4b24d 100644
--- a/Source/modules/mediastream/MediaStream.cpp
+++ b/Source/modules/mediastream/MediaStream.cpp
@@ -27,6 +27,9 @@
 #include "modules/mediastream/MediaStream.h"
 
 #include "bindings/core/v8/ExceptionState.h"
+#include "bindings/core/v8/ScriptValue.h"
+#include "bindings/modules/v8/V8MediaStream.h"
+#include "core/dom/DOMURL.h"
 #include "core/dom/ExceptionCode.h"
 #include "modules/mediastream/MediaStreamRegistry.h"
 #include "modules/mediastream/MediaStreamTrackEvent.h"
@@ -54,6 +57,12 @@ static void processTrack(MediaStreamTrack* track, MediaStreamTrackVector& trackV
         trackVector.append(track);
 }
 
+static MediaProvider* MediaProviderConverter(const ScriptValue& value)
+{
+    v8::Handle<v8::Value> v8Value(value.v8Value());
+    return V8MediaStream::toImplWithTypeCheck(value.isolate(), v8Value);

[haraken, 2014/09/20 08:52:20]

Hmm, I'm not really happy with writing this kind of binding logic in core/. Can
we move the conversion code into V8HTMLMediaElementCustom.cpp?

[perkj_chrome, 2014/09/24 11:39:39]

But can bindings/core/v8/V8HTMLMediaElementCustom 
really depend on  "bindings/modules/v8/V8MediaStream.h" ?

+}
+
 MediaStream* MediaStream::create(ExecutionContext* context)
 {
     MediaStreamTrackVector audioTracks;
@@ -95,7 +104,8 @@ MediaStream* MediaStream::create(ExecutionContext* context, PassRefPtr<MediaStre
 }
 
 MediaStream::MediaStream(ExecutionContext* context, PassRefPtr<MediaStreamDescriptor> streamDescriptor)
-    : ContextLifecycleObserver(context)
+    : MediaProvider(EventTargetNames::MediaStream, bind<const ScriptValue&>(MediaProviderConverter))
+    , ContextLifecycleObserver(context)
     , m_stopped(false)
     , m_descriptor(streamDescriptor)
     , m_scheduledEventTimer(this, &MediaStream::scheduledEventTimerFired)
@@ -120,7 +130,8 @@ MediaStream::MediaStream(ExecutionContext* context, PassRefPtr<MediaStreamDescri
 }
 
 MediaStream::MediaStream(ExecutionContext* context, const MediaStreamTrackVector& audioTracks, const MediaStreamTrackVector& videoTracks)
-    : ContextLifecycleObserver(context)
+    : MediaProvider(EventTargetNames::MediaStream, bind<const ScriptValue&>(MediaProviderConverter))
+    , ContextLifecycleObserver(context)
     , m_stopped(false)
     , m_scheduledEventTimer(this, &MediaStream::scheduledEventTimerFired)
 {
@@ -147,6 +158,8 @@ MediaStream::MediaStream(ExecutionContext* context, const MediaStreamTrackVector
 
 MediaStream::~MediaStream()
 {
+    if (!m_objectUrl.isEmpty())
+        DOMURL::revokeObjectURL(executionContext(), m_objectUrl);
     m_descriptor->setClient(0);
 }
 
@@ -387,11 +400,19 @@ URLRegistry& MediaStream::registry() const
     return MediaStreamRegistry::registry();
 }
 
+const String& MediaStream::getObjectUrl()
+{
+    if (m_objectUrl.isEmpty())
+        m_objectUrl = DOMURL::createPublicURL(executionContext(), this);
+    return m_objectUrl;
+}
+
 void MediaStream::trace(Visitor* visitor)
 {
     visitor->trace(m_audioTracks);
     visitor->trace(m_videoTracks);
     visitor->trace(m_scheduledEvents);
+    MediaProvider::trace(visitor);
     EventTargetWithInlineData::trace(visitor);
 }