Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(500)

Unified Diff: third_party/WebKit/Source/core/html/track/CueTimeline.cpp

Issue 1418133011: Remove the named item getters on TextTrackList and TextTrackCueList (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/html/track/CueTimeline.cpp
diff --git a/third_party/WebKit/Source/core/html/track/CueTimeline.cpp b/third_party/WebKit/Source/core/html/track/CueTimeline.cpp
index bab2ebbecc25038b92349b416c741250b5d9aff7..3fd14852385a7a95aca2592f530439523d9aec53 100644
--- a/third_party/WebKit/Source/core/html/track/CueTimeline.cpp
+++ b/third_party/WebKit/Source/core/html/track/CueTimeline.cpp
@@ -27,7 +27,7 @@ void CueTimeline::addCues(TextTrack* track, const TextTrackCueList* cues)
{
ASSERT(track->mode() != TextTrack::disabledKeyword());
for (size_t i = 0; i < cues->length(); ++i)
- addCueInternal(cues->item(i));
+ addCueInternal(cues->anonymousIndexedGetter(i));
updateActiveCues(mediaElement().currentTime());
}
@@ -52,7 +52,7 @@ void CueTimeline::addCueInternal(PassRefPtrWillBeRawPtr<TextTrackCue> cue)
void CueTimeline::removeCues(TextTrack*, const TextTrackCueList* cues)
{
for (size_t i = 0; i < cues->length(); ++i)
- removeCueInternal(cues->item(i));
+ removeCueInternal(cues->anonymousIndexedGetter(i));
updateActiveCues(mediaElement().currentTime());
}
@@ -86,7 +86,7 @@ void CueTimeline::removeCueInternal(PassRefPtrWillBeRawPtr<TextTrackCue> cue)
void CueTimeline::hideCues(TextTrack*, const TextTrackCueList* cues)
{
for (size_t i = 0; i < cues->length(); ++i)
- cues->item(i)->removeDisplayTree();
+ cues->anonymousIndexedGetter(i)->removeDisplayTree();
}
static bool trackIndexCompare(TextTrack* a, TextTrack* b)

Powered by Google App Engine
This is Rietveld 408576698