Bug 168800 - [Modern Media Controls] No controls are shown for <audio preload="none">
Summary: [Modern Media Controls] No controls are shown for <audio preload="none">
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Antoine Quint
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-02-23 14:25 PST by Antoine Quint
Modified: 2017-02-23 23:40 PST (History)
4 users (show)

See Also:


Attachments
Patch (8.40 KB, patch)
2017-02-23 17:09 PST, Antoine Quint
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews104 for mac-elcapitan-wk2 (962.72 KB, application/zip)
2017-02-23 18:16 PST, Build Bot
no flags Details
Archive of layout-test-results from ews115 for mac-elcapitan (1.93 MB, application/zip)
2017-02-23 19:19 PST, Build Bot
no flags Details
Patch for landing (11.17 KB, patch)
2017-02-23 22:58 PST, Antoine Quint
no flags Details | Formatted Diff | Diff
Patch for landing (10.19 KB, patch)
2017-02-23 23:00 PST, Antoine Quint
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Antoine Quint 2017-02-23 14:25:46 PST
We are currently not set to show ANY controls unless .currentSrc returns something other than an empty string. If a media element has the "src" attribute set but "preload" is set to "none" then loading isn’t triggered and .currentSrc isn’t resolved. We should always show the start button for <video> and the controls for <audio> in that case, and hope for the best, although the media may turn out to be invalid.
Comment 1 Antoine Quint 2017-02-23 14:25:59 PST
<rdar://problem/30652142>
Comment 2 Antoine Quint 2017-02-23 17:09:34 PST
Created attachment 302611 [details]
Patch
Comment 3 Build Bot 2017-02-23 18:16:51 PST
Comment on attachment 302611 [details]
Patch

Attachment 302611 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/3182208

New failing tests:
fast/regions/inline-block-inside-anonymous-overflow-with-covered-controls.html
Comment 4 Build Bot 2017-02-23 18:16:54 PST
Created attachment 302622 [details]
Archive of layout-test-results from ews104 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 5 Build Bot 2017-02-23 19:19:28 PST
Comment on attachment 302611 [details]
Patch

Attachment 302611 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/3182504

New failing tests:
media/modern-media-controls/volume-down-support/volume-down-support.html
Comment 6 Build Bot 2017-02-23 19:19:32 PST
Created attachment 302629 [details]
Archive of layout-test-results from ews115 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 7 Antoine Quint 2017-02-23 22:58:40 PST
Created attachment 302646 [details]
Patch for landing
Comment 8 WebKit Commit Bot 2017-02-23 22:59:45 PST
Comment on attachment 302646 [details]
Patch for landing

Rejecting attachment 302646 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'apply-attachment', '--no-update', '--non-interactive', 302646, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
-support-controls-on-audio-expected.txt
patching file LayoutTests/media/modern-media-controls/controls-visibility-support/controls-visibility-support-controls-on-audio.html
patching file LayoutTests/media/modern-media-controls/start-support/start-support-no-source-expected.txt
patching file LayoutTests/media/modern-media-controls/start-support/start-support-no-source.html

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: http://webkit-queues.webkit.org/results/3183512
Comment 9 Antoine Quint 2017-02-23 23:00:59 PST
Created attachment 302648 [details]
Patch for landing
Comment 10 Antoine Quint 2017-02-23 23:02:22 PST
This made a region test fail and it was set to use legacy media controls instead, see https://bugs.webkit.org/show_bug.cgi?id=168823.
Comment 11 WebKit Commit Bot 2017-02-23 23:40:30 PST
Comment on attachment 302648 [details]
Patch for landing

Clearing flags on attachment: 302648

Committed r212948: <http://trac.webkit.org/changeset/212948>
Comment 12 WebKit Commit Bot 2017-02-23 23:40:35 PST
All reviewed patches have been landed.  Closing bug.