Bug 168047 - [Modern Media Controls] Playing an <audio> element on iOS makes the controls disappear
Summary: [Modern Media Controls] Playing an <audio> element on iOS makes the controls ...
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
: 168048 (view as bug list)
Depends on:
Blocks:
 
Reported: 2017-02-09 06:29 PST by Antoine Quint
Modified: 2017-02-09 08:22 PST (History)
2 users (show)

See Also:


Attachments
Patch (8.23 KB, patch)
2017-02-09 07:27 PST, Antoine Quint
no flags Details | Formatted Diff | Diff
Patch for landing (8.23 KB, patch)
2017-02-09 07:50 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-09 06:29:28 PST
Playing an <audio> element on iOS makes the controls disappear
Comment 1 Antoine Quint 2017-02-09 06:30:03 PST
*** Bug 168048 has been marked as a duplicate of this bug. ***
Comment 2 Radar WebKit Bug Importer 2017-02-09 06:30:43 PST
<rdar://problem/30441939>
Comment 3 Antoine Quint 2017-02-09 07:27:16 PST
Created attachment 301035 [details]
Patch
Comment 4 Eric Carlson 2017-02-09 07:31:07 PST
Comment on attachment 301035 [details]
Patch

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

> Source/WebCore/ChangeLog:9
> +        The controls bar auot-hide behavior is disabled for <audio> elements, but on iOS we have

Nit: "auot" -> "auto"

> LayoutTests/media/modern-media-controls/media-controller/media-controller-ios-only-enable-tap-gesture-recognizer-with-fades-when-idle.html:8
> +description("Testing the <code>MediaController</code> controls bar tap gesture recognizer is only enabled when shouldBeFalse is true.");

"when shouldBeFalse is true"? 

Do you mean "when fadesWhileIdle is true"?
Comment 5 Antoine Quint 2017-02-09 07:50:52 PST
Created attachment 301038 [details]
Patch for landing
Comment 6 Antoine Quint 2017-02-09 08:21:57 PST
Committed r211955: <http://trac.webkit.org/changeset/211955>