Fix TextTrackCue::cueIndex() to handle the null case of TextTrack::cues(() properly
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 8 Jul 2013 18:33:32 +0000 (18:33 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 8 Jul 2013 18:33:32 +0000 (18:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=117815
<rdar://problem/14211041>

Patch by Ruth Fong <ruth_fong@apple.com> on 2013-07-08
Reviewed by Eric Carlson.

Merge https://src.chromium.org/viewvc/blink?revision=153206&view=revision

No new tests, only behavior change is preventing a crash in the release build.
cueIndex() is only used for sorting cues, so returning an invalidCueIndex in the case
where track()->cues() is null makes sense and won't modify behavior.

* html/track/TextTrackCue.cpp:
(WebCore::TextTrackCue::cueIndex):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@152459 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/html/track/TextTrackCue.cpp

index b619f2b..e6483b6 100644 (file)
@@ -1,3 +1,20 @@
+2013-07-08  Ruth Fong  <ruth_fong@apple.com>
+
+        Fix TextTrackCue::cueIndex() to handle the null case of TextTrack::cues(() properly
+        https://bugs.webkit.org/show_bug.cgi?id=117815
+        <rdar://problem/14211041>
+
+        Reviewed by Eric Carlson.
+
+        Merge https://src.chromium.org/viewvc/blink?revision=153206&view=revision
+        
+        No new tests, only behavior change is preventing a crash in the release build.
+        cueIndex() is only used for sorting cues, so returning an invalidCueIndex in the case
+        where track()->cues() is null makes sense and won't modify behavior.
+
+        * html/track/TextTrackCue.cpp:
+        (WebCore::TextTrackCue::cueIndex):
+
 2013-07-08  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r152416.
index 085fb06..67e39f6 100644 (file)
@@ -486,7 +486,8 @@ int TextTrackCue::cueIndex()
     if (m_cueIndex == invalidCueIndex) {
         ASSERT(track());
         ASSERT(track()->cues());
-        m_cueIndex = track()->cues()->getCueIndex(this);
+        if (TextTrackCueList* cueList = track()->cues())
+            m_cueIndex = cueList->getCueIndex(this);
     }
 
     return m_cueIndex;