|Summary:||Adding a text track should not make controls visible|
|Product:||WebKit||Reporter:||Eric Carlson <eric.carlson>|
|Component:||Media||Assignee:||Eric Carlson <eric.carlson>|
|Severity:||Normal||CC:||dglazkov, feature-media-reviews, ojan.autocc, webkit-bug-importer, webkit.review.bot|
|Version:||528+ (Nightly build)|
|Bug Depends on:|
Description Eric Carlson 2013-01-25 09:34:19 PST
Adding a text track makes controls visible
Comment 2 Eric Carlson 2013-01-25 12:13:30 PST
Created attachment 184791 [details] Updated patch. Updated patch as the first version sometimes creates hidden controls when we did not previously. This isn't really a change in functionality, but it causes some tests that dump the render tree to fail so attempt to restore the previous behavior. Not marking r? so the bots can chew on it first.
Comment 3 WebKit Review Bot 2013-01-25 12:58:46 PST
Comment on attachment 184791 [details] Updated patch. Attachment 184791 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/16122473 New failing tests: media/media-captions-no-controls.html
Comment 4 Eric Carlson 2013-01-25 13:19:03 PST
Created attachment 184801 [details] Updated patch. Oops, the results for my new test depended on the side effect I removed in the second patch :-O
Comment 5 Eric Carlson 2013-01-25 13:43:15 PST
Comment on attachment 184801 [details] Updated patch. Committed r140862.