[Webkit-unassigned] [Bug 117815] Fix TextTrackCue::cueIndex() to handle the null case of TextTrack::cues(() properly

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=117815





--- Comment #18 from Eric Carlson <eric.carlson at apple.com>  2013-07-15 07:43:46 PST ---
(In reply to comment #16)
> Why was test case not added? Did you try using test.mp4 instead of test.ogv?

Because the test case is worthless, see my comment here: https://chromiumcodereview.appspot.com/17002002#msg29


(In reply to comment #17)
> (In reply to comment #16)
> > Why was test case not added? Did you try using test.mp4 instead of test.ogv?
> 
> The test hits a case that isn't solved by this patch: https://chromiumcodereview.appspot.com/17002002#msg31
> 
> We should consider merging this updated patch: https://src.chromium.org/viewvc/blink?revision=153810&view=revision

Agreed.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.