Title: [186094] trunk/Source/WebCore
Revision
186094
Author
[email protected]
Date
2015-06-29 16:14:45 -0700 (Mon, 29 Jun 2015)

Log Message

Removing an element from a media session should also remove it from the sets of active participating elements
https://bugs.webkit.org/show_bug.cgi?id=146420

Reviewed by Eric Carlson.

* Modules/mediasession/MediaSession.cpp:
(WebCore::MediaSession::removeMediaElement):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (186093 => 186094)


--- trunk/Source/WebCore/ChangeLog	2015-06-29 23:08:20 UTC (rev 186093)
+++ trunk/Source/WebCore/ChangeLog	2015-06-29 23:14:45 UTC (rev 186094)
@@ -1,3 +1,13 @@
+2015-06-29  Matt Rajca  <[email protected]>
+
+        Removing an element from a media session should also remove it from the sets of active participating elements
+        https://bugs.webkit.org/show_bug.cgi?id=146420
+
+        Reviewed by Eric Carlson.
+
+        * Modules/mediasession/MediaSession.cpp:
+        (WebCore::MediaSession::removeMediaElement):
+
 2015-06-29  Dean Jackson  <[email protected]>
 
         Temporarily disable PICTURE_SIZES

Modified: trunk/Source/WebCore/Modules/mediasession/MediaSession.cpp (186093 => 186094)


--- trunk/Source/WebCore/Modules/mediasession/MediaSession.cpp	2015-06-29 23:08:20 UTC (rev 186093)
+++ trunk/Source/WebCore/Modules/mediasession/MediaSession.cpp	2015-06-29 23:14:45 UTC (rev 186094)
@@ -114,6 +114,10 @@
 {
     ASSERT(m_participatingElements.contains(&element));
     m_participatingElements.remove(&element);
+
+    m_activeParticipatingElements.remove(&element);
+    if (m_iteratedActiveParticipatingElements)
+        m_iteratedActiveParticipatingElements->remove(&element);
 }
 
 void MediaSession::addActiveMediaElement(HTMLMediaElement& element)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to