Title: [176322] trunk/Source/WebCore
Revision
176322
Author
[email protected]
Date
2014-11-19 09:52:58 -0800 (Wed, 19 Nov 2014)

Log Message

Missing NULL-check in VideoTrack::setLanguage
https://bugs.webkit.org/show_bug.cgi?id=138867

Reviewed by Jer Noble.

* html/track/VideoTrack.cpp:
(WebCore::VideoTrack::setLanguage): Prevent change event creation
on an empty video track list.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (176321 => 176322)


--- trunk/Source/WebCore/ChangeLog	2014-11-19 17:36:29 UTC (rev 176321)
+++ trunk/Source/WebCore/ChangeLog	2014-11-19 17:52:58 UTC (rev 176322)
@@ -1,3 +1,14 @@
+2014-11-19  Philippe Normand  <[email protected]>
+
+        Missing NULL-check in VideoTrack::setLanguage
+        https://bugs.webkit.org/show_bug.cgi?id=138867
+
+        Reviewed by Jer Noble.
+
+        * html/track/VideoTrack.cpp:
+        (WebCore::VideoTrack::setLanguage): Prevent change event creation
+        on an empty video track list.
+
 2014-11-19  Chris Fleizach  <[email protected]>
 
         AX: Safari fails to load a web page with VoiceOver

Modified: trunk/Source/WebCore/html/track/VideoTrack.cpp (176321 => 176322)


--- trunk/Source/WebCore/html/track/VideoTrack.cpp	2014-11-19 17:36:29 UTC (rev 176321)
+++ trunk/Source/WebCore/html/track/VideoTrack.cpp	2014-11-19 17:52:58 UTC (rev 176322)
@@ -217,7 +217,8 @@
 
     // 4. Queue a task to fire a simple event named change at the VideoTrackList object referenced by
     // the videoTracks attribute on the HTMLMediaElement.
-    mediaElement()->videoTracks()->scheduleChangeEvent();
+    if (mediaElement()->videoTracks())
+        mediaElement()->videoTracks()->scheduleChangeEvent();
 }
 #endif
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to