73405 2011-11-30 00:06:11 -0800 When playing audio in <video>, the poster is hidden on play 2011-12-01 09:01:56 -0800 1 1 1 Unclassified WebKit Media 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED http://goo.gl/0g77d InRadar P2 Normal --- 1 mail eric.carlson eric.carlson webkit-bug-importer oldest_to_newest 511448 0 mail 2011-11-30 00:06:11 -0800 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. 512048 1 eric.carlson 2011-11-30 13:01:56 -0800 Safari on 10.7.2 behaves correctly. 512050 2 eric.carlson 2011-11-30 13:04:38 -0800 But Safari 5.1 on 10.6.8 does not, so it apparently depends on the media engine. 512057 3 eric.carlson 2011-11-30 13:08:38 -0800 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. 512100 4 eric.carlson 2011-11-30 13:53:09 -0800 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. 512102 5 117265 eric.carlson 2011-11-30 13:53:53 -0800 Created attachment 117265 Proposed patch 512361 6 117265 darin 2011-11-30 19:35:14 -0800 Comment on attachment 117265 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 ;-) 512813 7 webkit-bug-importer 2011-12-01 09:01:07 -0800 <rdar://problem/10511668> 512815 8 eric.carlson 2011-12-01 09:01:56 -0800 http://trac.webkit.org/changeset/101675 117265 2011-11-30 13:53:53 -0800 2011-11-30 19:35:14 -0800 Proposed patch patch_poster_no_video.txt text/plain 1276 eric.carlson SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEwMTU1MCkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDExLTExLTMwICBFcmljIENh cmxzb24gIDxlcmljLmNhcmxzb25AYXBwbGUuY29tPgorCisgICAgICAgIFdoZW4gcGxheWluZyBh dWRpbyBpbiA8dmlkZW8+LCB0aGUgcG9zdGVyIGlzIGhpZGRlbiBvbiBwbGF5CisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD03MzQwNQorCisgICAgICAgIFJl dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIE5vIG5ldyB0ZXN0cy4gKE9PUFMh KQorCisgICAgICAgICogaHRtbC9IVE1MVmlkZW9FbGVtZW50LmNwcDoKKyAgICAgICAgKFdlYkNv cmU6OkhUTUxWaWRlb0VsZW1lbnQ6Omhhc0F2YWlsYWJsZVZpZGVvRnJhbWUpOiBEb24ndCByZXR1 cm4gdHJ1ZSBpZiB0aGUgZmlsZQorICAgICAgICAgICAgZG9lc24ndCBoYXZlIHZpZGVvLgorCiAy MDExLTExLTMwICBEYXZpZCBMZXZpbiAgPGxldmluQGNocm9taXVtLm9yZz4KIAogICAgICAgICBN YWtlIEZyYW1lVmlldyB1c2UgVGVtcG9yYXJpbHlDaGFuZ2UgaW4gYSBmZXcgcGxhY2VzLgpJbmRl eDogU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MVmlkZW9FbGVtZW50LmNwcAo9PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t LSBTb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxWaWRlb0VsZW1lbnQuY3BwCShyZXZpc2lvbiAxMDEy NjMpCisrKyBTb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxWaWRlb0VsZW1lbnQuY3BwCSh3b3JraW5n IGNvcHkpCkBAIC0yMzcsNyArMjM3LDcgQEAgYm9vbCBIVE1MVmlkZW9FbGVtZW50OjpoYXNBdmFp bGFibGVWaWRlbwogICAgIGlmICghcGxheWVyKCkpCiAgICAgICAgIHJldHVybiBmYWxzZTsKICAg ICAKLSAgICByZXR1cm4gcGxheWVyKCktPmhhc0F2YWlsYWJsZVZpZGVvRnJhbWUoKTsKKyAgICBy ZXR1cm4gcGxheWVyKCktPmhhc1ZpZGVvKCkgJiYgcGxheWVyKCktPmhhc0F2YWlsYWJsZVZpZGVv RnJhbWUoKTsKIH0KIAogdm9pZCBIVE1MVmlkZW9FbGVtZW50Ojp3ZWJraXRFbnRlckZ1bGxzY3Jl ZW4oRXhjZXB0aW9uQ29kZSYgZWMpCg==