62036 2011-06-03 12:01:01 -0700 Video with poster attribute will not display new frames when scrubbing. (AVFoundation) 2011-06-07 16:02:36 -0700 1 1 1 Unclassified WebKit Media 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar, PlatformOnly P2 Normal --- 1 jer.noble jer.noble eric.carlson webkit.review.bot oldest_to_newest 415065 0 jer.noble 2011-06-03 12:01:01 -0700 Video with poster attribute will not display new frames when scrubbing. 415069 1 jer.noble 2011-06-03 12:02:33 -0700 This bug only reproduces on Lion using AVFoundation. 1. Create a html document and a video element with the format: <video src="movie" poster="poster" controls></video> 2. Load the page in Safari 3. Drag the timeline slider 4. Frame stays at the poster image. 415070 2 jer.noble 2011-06-03 12:02:38 -0700 <rdar://problem/9550024> 415071 3 jer.noble 2011-06-03 12:02:47 -0700 Looks like a timing issue. AVFoundation is firing our seekToTime: completion block handler before our periodic time observer. By the time the timeChanged notification in handled, our m_seekTo variable has already been reset, and the notification is dropped. 415123 4 95953 jer.noble 2011-06-03 13:16:36 -0700 Created attachment 95953 Patch 416814 5 95953 adele 2011-06-07 15:04:59 -0700 Comment on attachment 95953 Patch Marking cq+ to expedite this! 416885 6 95953 webkit.review.bot 2011-06-07 16:02:32 -0700 Comment on attachment 95953 Patch Clearing flags on attachment: 95953 Committed r88279: <http://trac.webkit.org/changeset/88279> 416886 7 webkit.review.bot 2011-06-07 16:02:36 -0700 All reviewed patches have been landed. Closing bug. 95953 2011-06-03 13:16:36 -0700 2011-06-07 16:02:32 -0700 Patch bug-62036-20110603131635.patch text/plain 2701 jer.noble U3VidmVyc2lvbiBSZXZpc2lvbjogODgwNDEKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCA3ODBmMDEwNDhkOTdkYjBh NThhMGE1N2U1MDdjOWYyYTU3MGNjYjRiLi40NzhkZmNjOTk2MjQ4MDdiMTBjNTAyYTQyYjlmYzk4 OTMzNDY2OWNkIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291 cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTggQEAKKzIwMTEtMDYtMDMgIEplciBO b2JsZSAgPGplci5ub2JsZUBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgVmlkZW8gd2l0aCBwb3N0ZXIgYXR0cmlidXRlIHdpbGwgbm90 IGRpc3BsYXkgbmV3IGZyYW1lcyB3aGVuIHNjcnViYmluZy4KKyAgICAgICAgaHR0cHM6Ly9idWdz LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTYyMDM2CisKKyAgICAgICAgUmVtb3ZlIG91ciAi ZGV0ZWN0aW9uIiBhYm91dCB0aGUgZW5kIG9mIGEgc2NydWIgbm93IHRoYXQgdGhlcmUgaXMgYW4g ZXhwbGljaXQgc2VlayBjb21wbGV0aW9uCisgICAgICAgIGhhbmRsZXIgYXZhaWxhYmxlIGluIEFW Rm91bmRhdGlvbi4gIE1vdmUgd2hhdCB3ZSB1c2VkIHRvIGRvIHVwb24gZGV0ZWN0aW9uIGluIHRp bWVDaGFuZ2VkKCkKKyAgICAgICAgaW50byBzZWVrQ29tcGxldGVkKCkuCisKKyAgICAgICAgKiBw bGF0Zm9ybS9ncmFwaGljcy9hdmZvdW5kYXRpb24vTWVkaWFQbGF5ZXJQcml2YXRlQVZGb3VuZGF0 aW9uLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6Ok1lZGlhUGxheWVyUHJpdmF0ZUFWRm91bmRhdGlv bjo6dGltZUNoYW5nZWQpOiAKKyAgICAgICAgKFdlYkNvcmU6Ok1lZGlhUGxheWVyUHJpdmF0ZUFW Rm91bmRhdGlvbjo6c2Vla0NvbXBsZXRlZCk6CisKIDIwMTEtMDYtMDMgIFJvYiBCdWlzICA8cmJ1 aXNAcmltLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBOaWtvbGFzIFppbW1lcm1hbm4uCmRp ZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9hdmZvdW5kYXRpb24v TWVkaWFQbGF5ZXJQcml2YXRlQVZGb3VuZGF0aW9uLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsYXRm b3JtL2dyYXBoaWNzL2F2Zm91bmRhdGlvbi9NZWRpYVBsYXllclByaXZhdGVBVkZvdW5kYXRpb24u Y3BwCmluZGV4IDAzMzYwNzdjM2Q3NTBiZWNmNjFlZTMwZjZkNWQ3YTJjOTU3ZjdhZDUuLjNjM2Ez ZjA0NmQ5OGMzNWY0YWUwNTI0NmNlMTQxZjM3NjgzMWQzMzAgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X ZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2F2Zm91bmRhdGlvbi9NZWRpYVBsYXllclByaXZhdGVB VkZvdW5kYXRpb24uY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2F2 Zm91bmRhdGlvbi9NZWRpYVBsYXllclByaXZhdGVBVkZvdW5kYXRpb24uY3BwCkBAIC01NTEsMjkg KzU1MSwxNSBAQCB2b2lkIE1lZGlhUGxheWVyUHJpdmF0ZUFWRm91bmRhdGlvbjo6c2Vla2FibGVU aW1lUmFuZ2VzQ2hhbmdlZCgpCiB2b2lkIE1lZGlhUGxheWVyUHJpdmF0ZUFWRm91bmRhdGlvbjo6 dGltZUNoYW5nZWQoZG91YmxlIHRpbWUpCiB7CiAgICAgTE9HKE1lZGlhLCAiTWVkaWFQbGF5ZXJQ cml2YXRlQVZGb3VuZGF0aW9uOjp0aW1lQ2hhbmdlZCglcCkgLSB0aW1lID0gJWYiLCB0aGlzLCB0 aW1lKTsKLQotICAgIGlmIChtX3NlZWtUbyA9PSBpbnZhbGlkVGltZSgpKQotICAgICAgICByZXR1 cm47Ci0KLSAgICAvLyBBVkZvdW5kYXRpb24gbWF5IGNhbGwgb3VyIG9ic2VydmVyIG1vcmUgdGhh biBvbmNlIGR1cmluZyBhIHNlZWssIGFuZCB3ZSBjYW4ndCBjdXJyZW50bHkgdGVsbAotICAgIC8v IGlmIHdlIHdpbGwgYmUgYWJsZSB0byBzZWVrIHRvIGFuIGV4YWN0IHRpbWUsIHNvIGFzc3VtZSB0 aGF0IHdlIGFyZSBkb25lIHNlZWtpbmcgaWYgd2UgYXJlCi0gICAgLy8gImNsb3NlIGVub3VnaCIg dG8gdGhlIHNlZWsgdGltZS4KLSAgICBjb25zdCBkb3VibGUgc21hbGxTZWVrRGVsdGEgPSAxLjAg LyAxMDA7Ci0KLSAgICBmbG9hdCBjdXJyZW50UmF0ZSA9IHJhdGUoKTsKLSAgICBpZiAoKGN1cnJl bnRSYXRlID4gMCAmJiB0aW1lID49IG1fc2Vla1RvKSB8fCAoY3VycmVudFJhdGUgPCAwICYmIHRp bWUgPD0gbV9zZWVrVG8pIHx8IChhYnMobV9zZWVrVG8gLSB0aW1lKSA8PSBzbWFsbFNlZWtEZWx0 YSkpIHsKLSAgICAgICAgbV9zZWVrVG8gPSBpbnZhbGlkVGltZSgpOwotICAgICAgICB1cGRhdGVT dGF0ZXMoKTsKLSAgICAgICAgbV9wbGF5ZXItPnRpbWVDaGFuZ2VkKCk7Ci0gICAgfQogfQogCiB2 b2lkIE1lZGlhUGxheWVyUHJpdmF0ZUFWRm91bmRhdGlvbjo6c2Vla0NvbXBsZXRlZChib29sIGZp bmlzaGVkKQogewogICAgIExPRyhNZWRpYSwgIk1lZGlhUGxheWVyUHJpdmF0ZUFWRm91bmRhdGlv bjo6c2Vla0NvbXBsZXRlZCglcCkgLSBmaW5pc2hlZCA9ICVkIiwgdGhpcywgZmluaXNoZWQpOwog ICAgIAotICAgIGlmIChmaW5pc2hlZCkKLSAgICAgICAgbV9zZWVrVG8gPSBpbnZhbGlkVGltZSgp OworICAgIG1fc2Vla1RvID0gaW52YWxpZFRpbWUoKTsKKyAgICB1cGRhdGVTdGF0ZXMoKTsKKyAg ICBtX3BsYXllci0+dGltZUNoYW5nZWQoKTsKIH0KIAogdm9pZCBNZWRpYVBsYXllclByaXZhdGVB VkZvdW5kYXRpb246OmRpZEVuZCgpCg==