Support text track selection in video controls

third_party/WebKit/Source/core/html/track/TextTrack.cpp

Index: third_party/WebKit/Source/core/html/track/TextTrack.cpp
diff --git a/third_party/WebKit/Source/core/html/track/TextTrack.cpp b/third_party/WebKit/Source/core/html/track/TextTrack.cpp
index 1cd60174bdb63963e3b387e178c1848c9b1bfd7f..d97bbc9eb8df4827568bf714d4e841b298140090 100644
--- a/third_party/WebKit/Source/core/html/track/TextTrack.cpp
+++ b/third_party/WebKit/Source/core/html/track/TextTrack.cpp
@@ -443,6 +443,14 @@ int TextTrack::trackIndexRelativeToRenderedTracks()
     return m_renderedTrackIndex;
 }
 
+bool TextTrack::isRenderable()

[fs, 2016/02/23 13:16:44]

Maybe move this closer to isRendered() and format it similarly so that it
becomes easy to see what they have in common and how they differ? Maybe it
should be "canBeRendered()" to avoid it looking to similar to isRendered() (and
to provide some form of relation between them)?

[srivats, 2016/02/24 05:20:18]

Done.

+{
+    // A track can be displayed when it's of kind captions or subtitles and hasn't failed to load

[fs, 2016/02/23 13:16:44]

uNit: Add full stop.

[srivats, 2016/02/24 05:20:18]

Done.

+    if ((kind() == captionsKeyword() || kind() == subtitlesKeyword()) && readinessState() != FailedToLoad)
+        return true;
+    return false;
+}
+
 const AtomicString& TextTrack::interfaceName() const
 {
     return EventTargetNames::TextTrack;