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

Unified Diff: Source/core/html/track/TextTrackCue.cpp

Issue 18856005: Fix HTMLMediaElement so that it doesn't add cues for disabled text tracks. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 5 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: Source/core/html/track/TextTrackCue.cpp
diff --git a/Source/core/html/track/TextTrackCue.cpp b/Source/core/html/track/TextTrackCue.cpp
index b3263fd0d4123e90cc1e4879c5585e5198942b0d..2469c8a342a3b306f6dbb8ab82f83d6c15cb27c7 100644
--- a/Source/core/html/track/TextTrackCue.cpp
+++ b/Source/core/html/track/TextTrackCue.cpp
@@ -479,11 +479,8 @@ void TextTrackCue::setText(const String& text)
int TextTrackCue::cueIndex()
{
- if (m_cueIndex == invalidCueIndex) {
- TextTrackCueList* cues = track()->cues();
- if (cues)
- m_cueIndex = cues->getCueIndex(this);
- }
+ if (m_cueIndex == invalidCueIndex)
+ m_cueIndex = track()->cues()->getCueIndex(this);
return m_cueIndex;
}
@@ -1191,4 +1188,3 @@ bool TextTrackCue::operator==(const TextTrackCue& cue) const
}
} // namespace WebCore
-
« Source/core/html/HTMLMediaElement.cpp ('K') | « Source/core/html/track/LoadableTextTrack.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698