MediaStream should fire ended event when all tracks are ended

Source/WebCore/Modules/mediastream/MediaStreamTrack.h

Index: Source/WebCore/Modules/mediastream/MediaStreamTrack.h
diff --git a/Source/WebCore/Modules/mediastream/MediaStreamTrack.h b/Source/WebCore/Modules/mediastream/MediaStreamTrack.h
index e7fd13e65b2f6752d6eb175cce5478c1070374be..8284745bd073079281340cc7e7dc39f06b961c04 100644
--- a/Source/WebCore/Modules/mediastream/MediaStreamTrack.h
+++ b/Source/WebCore/Modules/mediastream/MediaStreamTrack.h
@@ -39,11 +39,12 @@
 
 namespace WebCore {
 
+class MediaStream;
 class MediaStreamComponent;
 
 class MediaStreamTrack : public RefCounted<MediaStreamTrack>, public ActiveDOMObject, public EventTarget, public MediaStreamSource::Observer {
 public:
-    static PassRefPtr<MediaStreamTrack> create(ScriptExecutionContext*, MediaStreamComponent*);
+    static PassRefPtr<MediaStreamTrack> create(ScriptExecutionContext*, MediaStreamComponent*, MediaStream*);
     virtual ~MediaStreamTrack();
 
     String kind() const;
@@ -73,7 +74,7 @@ public:
     using RefCounted<MediaStreamTrack>::deref;
 
 private:
-    MediaStreamTrack(ScriptExecutionContext*, MediaStreamComponent*);
+    MediaStreamTrack(ScriptExecutionContext*, MediaStreamComponent*, MediaStream*);
 
     // EventTarget
     virtual EventTargetData* eventTargetData() OVERRIDE;
@@ -87,6 +88,7 @@ private:
 
     bool m_stopped;
     RefPtr<MediaStreamComponent> m_component;
+    MediaStream* m_stream;

[abarth-chromium, 2013/04/08 06:24:21]

What's the ownership relationship between MediaStreamTrack and m_stream?  They
both appear to be refcounted, so it's not clear to me what prevents m_stream
from pointing to freed memory.

 };
 
 typedef Vector<RefPtr<MediaStreamTrack> > MediaStreamTrackVector;