WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
73405
When playing audio in <video>, the poster is hidden on play
https://bugs.webkit.org/show_bug.cgi?id=73405
Summary
When playing audio in <video>, the poster is hidden on play
Jeroen Wijering
Reported
2011-11-30 00:06:11 PST
Playing an audio file in <video> is allowed and works fine. However, when the audio starts playing, the poster image is removed. Per the spec, the poster image should stay visible: "When no video data is available (... the media resource does not have a video channel), the video element represents the poster frame." This also provides a better experience. A poster image is nicer than a white screen.
Attachments
Proposed patch
(1.25 KB, patch)
2011-11-30 13:53 PST
,
Eric Carlson
darin
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Eric Carlson
Comment 1
2011-11-30 13:01:56 PST
Safari on 10.7.2 behaves correctly.
Eric Carlson
Comment 2
2011-11-30 13:04:38 PST
But Safari 5.1 on 10.6.8 does not, so it apparently depends on the media engine.
Eric Carlson
Comment 3
2011-11-30 13:08:38 PST
We display the poster until HTMLVideoElement::hasAvailableVideoFrame returns true. That method asks the media engine if has video to display. The QTKit based method (MediaPlayerPrivateQTKit::hasAvailableVideoFrame) can't tell if a QTMovieLayer actually has frames ready to display, but it also doesn't check to see if the movie has video at all.
Eric Carlson
Comment 4
2011-11-30 13:53:09 PST
This could be fixed in each of the media engines that have a problem (at least QTKit, QuickTime, and Chrome), or by adding one test to HTMLVideoElement. I am going to go with the later.
Eric Carlson
Comment 5
2011-11-30 13:53:53 PST
Created
attachment 117265
[details]
Proposed patch
Darin Adler
Comment 6
2011-11-30 19:35:14 PST
Comment on
attachment 117265
[details]
Proposed patch View in context:
https://bugs.webkit.org/attachment.cgi?id=117265&action=review
> Source/WebCore/ChangeLog:8 > + No new tests. (OOPS!)
Bad to not have tests. Also can’t land the patch without removing the OOPS ;-)
Radar WebKit Bug Importer
Comment 7
2011-12-01 09:01:07 PST
<
rdar://problem/10511668
>
Eric Carlson
Comment 8
2011-12-01 09:01:56 PST
http://trac.webkit.org/changeset/101675
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug