Summary: | [GTK] media controls is always displayed when video has a text track | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Wonchul Lee <wonchul.lee> | ||||||||
Component: | Media | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bugs-noreply, commit-queue, mcatanzaro | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Local Build | ||||||||||
Hardware: | Other | ||||||||||
OS: | Linux | ||||||||||
Attachments: |
|
Description
Wonchul Lee
2015-12-22 04:35:36 PST
Created attachment 268953 [details]
Patch
Comment on attachment 268953 [details]
Patch
Thanks for this patch. The code changes look sane, but I think this needs to be committed with a layout test, unless there's some reason it would be hard to test.
Created attachment 269254 [details]
Patch
@Michael, thanks for reviewing it, I updated some layout test what checked media controls not displaying on video with text track without 'controls' attribute by default and showing media controls on fullscreen regardless of 'controls' attribute. Comment on attachment 269254 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=269254&action=review Thanks Wonchul. Let's just rephrase the English text a bit: > LayoutTests/media/video-controls-no-display-with-text-track-expected.txt:1 > +Tests that not showing video controls with text track without controls attribute by default and displaying it when request fullscreen How about: "Tests that video controls are not shown for videos with a text track when the controls attribute is not present, except when fullscreen." Created attachment 269671 [details]
Patch
Thanks, I rephrased it as you mentioned. Comment on attachment 269671 [details] Patch Clearing flags on attachment: 269671 Committed r195511: <http://trac.webkit.org/changeset/195511> All reviewed patches have been landed. Closing bug. |