Bug 122468 - Track controls show metadata and chapter tracks
Summary: Track controls show metadata and chapter tracks
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Brendan Long
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-10-07 15:08 PDT by Brendan Long
Modified: 2013-10-10 23:14 PDT (History)
9 users (show)

See Also:


Attachments
Only show captions, descriptions and subtitles in the track menu (6.24 KB, patch)
2013-10-07 15:11 PDT, Brendan Long
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-13 for mac-mountainlion-wk2 (473.97 KB, application/zip)
2013-10-07 15:58 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion (478.92 KB, application/zip)
2013-10-07 16:18 PDT, Build Bot
no flags Details
Skip on Mac, Windows and EFL (8.92 KB, patch)
2013-10-09 09:45 PDT, Brendan Long
no flags Details | Formatted Diff | Diff
Replace RefPtr<TextTrack> with TextTrack* (8.90 KB, patch)
2013-10-09 10:06 PDT, Brendan Long
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brendan Long 2013-10-07 15:08:27 PDT
Track controls show metadata and chapter tracks
Comment 1 Brendan Long 2013-10-07 15:11:05 PDT
Created attachment 213619 [details]
Only show captions, descriptions and subtitles in the track menu

It doesn't make any sense to show chapters and metadata in the track menu. This patch doesn't show them.
Comment 2 Build Bot 2013-10-07 15:58:42 PDT
Comment on attachment 213619 [details]
Only show captions, descriptions and subtitles in the track menu

Attachment 213619 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/3717019

New failing tests:
media/video-controls-captions-trackmenu-only-captions-descriptions-and-subtitles.html
Comment 3 Build Bot 2013-10-07 15:58:43 PDT
Created attachment 213624 [details]
Archive of layout-test-results from webkit-ews-13 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-13  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 4 Darin Adler 2013-10-07 16:13:52 PDT
Comment on attachment 213619 [details]
Only show captions, descriptions and subtitles in the track menu

View in context: https://bugs.webkit.org/attachment.cgi?id=213619&action=review

The test is failing on the bot; you will need to figure out why before landing.

> Source/WebCore/page/CaptionUserPreferences.cpp:183
> +        RefPtr<TextTrack> track = trackList->item(i);

It’s not necessary to churn the reference counts of these tracks by putting them in a RefPtr. A raw pointer is a better choice. The best choice is to just use "auto".
Comment 5 Build Bot 2013-10-07 16:18:34 PDT
Comment on attachment 213619 [details]
Only show captions, descriptions and subtitles in the track menu

Attachment 213619 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/3713030

New failing tests:
media/video-controls-captions-trackmenu-only-captions-descriptions-and-subtitles.html
Comment 6 Build Bot 2013-10-07 16:18:36 PDT
Created attachment 213628 [details]
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-06  Port: mac-mountainlion  Platform: Mac OS X 10.8.5
Comment 7 Brendan Long 2013-10-08 15:35:17 PDT
The problem here is that all of the track menu tests are skipped on Mac. It looks like for some reason it can't find the caption button. I've had an ongoing problem where DumpRenderTree crashes on my Mac, so I'm having trouble tracking it down.
Comment 8 Brendan Long 2013-10-09 09:45:39 PDT
Created attachment 213787 [details]
Skip on Mac, Windows and EFL

EFL uses the caption button instead of the track menu. The trackmenu tests all fail on Mac right now (it can't find the button for some reason?), and Windows too (no canplaythrough event?). When the other tests are fixed, this one should start working too, so I just grouped it with them in the TestExpectations.
Comment 9 Brendan Long 2013-10-09 10:06:17 PDT
Created attachment 213790 [details]
Replace RefPtr<TextTrack> with TextTrack*
Comment 10 WebKit Commit Bot 2013-10-10 23:14:53 PDT
Comment on attachment 213790 [details]
Replace RefPtr<TextTrack> with TextTrack*

Clearing flags on attachment: 213790

Committed r157290: <http://trac.webkit.org/changeset/157290>
Comment 11 WebKit Commit Bot 2013-10-10 23:14:56 PDT
All reviewed patches have been landed.  Closing bug.