Title: [155364] trunk/Source/WebCore
Revision
155364
Author
[email protected]
Date
2013-09-09 11:19:47 -0700 (Mon, 09 Sep 2013)

Log Message

MediaStream API: Stop means stop
https://bugs.webkit.org/show_bug.cgi?id=120873

Reviewed by Jer Noble.

Merge blink https://chromium.googlesource.com/chromium/blink/+/9a97b0f3892320fa5741485e06a78183b77c3635
by Tommy Widenflycht.

Removing notification functionality from MediaStreamTrack::stop because the world is being torn down.

No new tests, the original author was unable to come up with a reliable test case.

* Modules/mediastream/MediaStreamTrack.cpp:
(WebCore::MediaStreamTrack::stop): Don't call didEndTrack, the world is going away.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (155363 => 155364)


--- trunk/Source/WebCore/ChangeLog	2013-09-09 17:56:35 UTC (rev 155363)
+++ trunk/Source/WebCore/ChangeLog	2013-09-09 18:19:47 UTC (rev 155364)
@@ -1,3 +1,20 @@
+2013-09-09  Eric Carlson  <[email protected]>
+
+        MediaStream API: Stop means stop
+        https://bugs.webkit.org/show_bug.cgi?id=120873
+
+        Reviewed by Jer Noble.
+
+        Merge blink https://chromium.googlesource.com/chromium/blink/+/9a97b0f3892320fa5741485e06a78183b77c3635
+        by Tommy Widenflycht.
+
+        Removing notification functionality from MediaStreamTrack::stop because the world is being torn down.
+
+        No new tests, the original author was unable to come up with a reliable test case.
+
+        * Modules/mediastream/MediaStreamTrack.cpp:
+        (WebCore::MediaStreamTrack::stop): Don't call didEndTrack, the world is going away.
+
 2013-09-09  Filip Pizlo  <[email protected]>
 
         Unreviewed, fix build.

Modified: trunk/Source/WebCore/Modules/mediastream/MediaStreamTrack.cpp (155363 => 155364)


--- trunk/Source/WebCore/Modules/mediastream/MediaStreamTrack.cpp	2013-09-09 17:56:35 UTC (rev 155363)
+++ trunk/Source/WebCore/Modules/mediastream/MediaStreamTrack.cpp	2013-09-09 18:19:47 UTC (rev 155364)
@@ -157,7 +157,6 @@
 void MediaStreamTrack::stop()
 {
     m_stopped = true;
-    didEndTrack();
 }
 
 const AtomicString& MediaStreamTrack::interfaceName() const
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to