Title: [152540] trunk/Source/WebCore
Revision
152540
Author
[email protected]
Date
2013-07-10 11:42:39 -0700 (Wed, 10 Jul 2013)

Log Message

Fix a bug in MediaStreamDescriptor::removeVideoComponent
https://bugs.webkit.org/show_bug.cgi?id=118446

Patch by Nick Diego Yamane <[email protected]> on 2013-07-10
Reviewed by Eric Carlson.

MediaStreamDescriptor::removeVideoComponent is currently trying to
remove the 'videoComponent' from the wrong vector.

* platform/mediastream/MediaStreamDescriptor.h:
(WebCore::MediaStreamDescriptor::removeVideoComponent):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (152539 => 152540)


--- trunk/Source/WebCore/ChangeLog	2013-07-10 18:34:19 UTC (rev 152539)
+++ trunk/Source/WebCore/ChangeLog	2013-07-10 18:42:39 UTC (rev 152540)
@@ -1,3 +1,16 @@
+2013-07-10  Nick Diego Yamane  <[email protected]>
+
+        Fix a bug in MediaStreamDescriptor::removeVideoComponent
+        https://bugs.webkit.org/show_bug.cgi?id=118446
+
+        Reviewed by Eric Carlson.
+
+        MediaStreamDescriptor::removeVideoComponent is currently trying to
+        remove the 'videoComponent' from the wrong vector.
+
+        * platform/mediastream/MediaStreamDescriptor.h:
+        (WebCore::MediaStreamDescriptor::removeVideoComponent):
+
 2013-07-10  Mario Sanchez Prada  <[email protected]>
 
         AX: Allow requesting the full plain text for an object with textUnderElement()

Modified: trunk/Source/WebCore/platform/mediastream/MediaStreamDescriptor.h (152539 => 152540)


--- trunk/Source/WebCore/platform/mediastream/MediaStreamDescriptor.h	2013-07-10 18:34:19 UTC (rev 152539)
+++ trunk/Source/WebCore/platform/mediastream/MediaStreamDescriptor.h	2013-07-10 18:42:39 UTC (rev 152540)
@@ -86,9 +86,9 @@
     void addVideoComponent(PassRefPtr<MediaStreamComponent> component) { m_videoComponents.append(component); }
     void removeVideoComponent(MediaStreamComponent* component)
     {
-        size_t pos = m_audioComponents.find(component);
+        size_t pos = m_videoComponents.find(component);
         if (pos != notFound)
-            m_audioComponents.remove(pos);
+            m_videoComponents.remove(pos);
     }
 
     bool ended() const { return m_ended; }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to