Bug 143322 - Media controls should not always auto-hide
Summary: Media controls should not always auto-hide
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Carlson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-04-01 15:35 PDT by Eric Carlson
Modified: 2015-04-02 06:55 PDT (History)
2 users (show)

See Also:


Attachments
Proposed patch (4.49 KB, patch)
2015-04-01 16:11 PDT, Eric Carlson
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Carlson 2015-04-01 15:35:29 PDT
Don't hide controls for an <audio> element, when playing to an external device, or for optimized fullscreen.
Comment 1 Eric Carlson 2015-04-01 15:43:41 PDT
<rdar://problem/20013740>
Comment 2 Radar WebKit Bug Importer 2015-04-01 15:44:24 PDT
<rdar://problem/20391156>
Comment 3 Eric Carlson 2015-04-01 16:11:00 PDT
Created attachment 249951 [details]
Proposed patch
Comment 4 Dean Jackson 2015-04-01 16:32:52 PDT
Comment on attachment 249951 [details]
Proposed patch

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

> Source/WebCore/ChangeLog:15
> +            and video.webkitWirelessVideoPlaybackDisabled because know when they change and use them

Typo: because *we* know

> Source/WebCore/Modules/mediacontrols/mediaControlsApple.js:18
> +    this.currrentTargetIsWireless = false;

Typo: currrent
Comment 5 Eric Carlson 2015-04-01 16:46:05 PDT
(In reply to comment #4)
> Comment on attachment 249951 [details]
> Proposed patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=249951&action=review
> 
> > Source/WebCore/ChangeLog:15
> > +            and video.webkitWirelessVideoPlaybackDisabled because know when they change and use them
> 
> Typo: because *we* know
> 
Fixed.

> > Source/WebCore/Modules/mediacontrols/mediaControlsApple.js:18
> > +    this.currrentTargetIsWireless = false;
> 
> Typo: currrent
>
Fixed.

Thanks!
Comment 6 Eric Carlson 2015-04-01 16:46:30 PDT
Committed r182262: https://trac.webkit.org/r182262