RESOLVED FIXED 133590
Automatic text track selection can select more than one track
https://bugs.webkit.org/show_bug.cgi?id=133590
Summary Automatic text track selection can select more than one track
Eric Carlson
Reported 2014-06-06 16:39:18 PDT
If the automatic text track selection logic chooses a track that is not in the user's preferred language, adding a new track in the user's selected language should disable the first track.
Attachments
Proposed patch. (10.58 KB, patch)
2014-06-06 17:06 PDT, Eric Carlson
no flags
Eric Carlson
Comment 1 2014-06-06 17:06:15 PDT
Created attachment 232637 [details] Proposed patch.
Brent Fulgham
Comment 2 2014-06-06 17:21:00 PDT
Comment on attachment 232637 [details] Proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=232637&action=review r=me > Source/WebCore/page/CaptionUserPreferencesMediaAF.cpp:774 > + if (kind != TextTrack::captionsKeyword() && kind != TextTrack::descriptionsKeyword() && kind != TextTrack::subtitlesKeyword()) This pattern gets used in a few places. Someday we should make some kind of "TextTrack::shouldDisplayKind(kind)" or something.
Radar WebKit Bug Importer
Comment 3 2014-06-09 15:27:36 PDT
WebKit Commit Bot
Comment 4 2014-06-09 15:57:11 PDT
Comment on attachment 232637 [details] Proposed patch. Clearing flags on attachment: 232637 Committed r169725: <http://trac.webkit.org/changeset/169725>
WebKit Commit Bot
Comment 5 2014-06-09 15:57:16 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.