26482 2009-06-17 10:00:31 -0700 Media element doesn't resume playing after playbackRate set to 0 2009-06-17 11:28:47 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 eric.carlson webkit-unassigned oldest_to_newest 126480 0 eric.carlson 2009-06-17 10:00:31 -0700 When the playbackRate of a media element that is currently playing is set to 0, playback stops and "paused" returns false (good so far). Changing the playbackRate to a non-zero value should resume playback, but it does not. * STEPS TO REPRODUCE 1. Open the attached example 2. Provide an audio file and change the src attribute on the audio element 3. Start playing your audio sample. 4. Play with the slider to change the playbackRate. 5. Move the slider from positive to negative, passing through zero. 6. Notice that once you've passed through zero, to -1 for example, the element does not continue playing. It should! 126481 1 31419 eric.carlson 2009-06-17 10:00:55 -0700 Created attachment 31419 test case 126483 2 eric.carlson 2009-06-17 10:01:30 -0700 <rdar://problem/6978590> 126489 3 31421 eric.carlson 2009-06-17 10:56:19 -0700 Created attachment 31421 proposed patch 126506 4 eric.carlson 2009-06-17 11:28:47 -0700 http://trac.webkit.org/changeset/44769 31419 2009-06-17 10:00:55 -0700 2009-06-17 10:00:55 -0700 test case test.html text/html 1297 eric.carlson PGh0bWw+Cgo8c2NyaXB0PgoKdmFyIG11c2ljTm9kZSA9IHVuZGVmaW5lZDsKdmFyIHJhdGVOb2Rl ID0gdW5kZWZpbmVkOwp2YXIgc2xpZGVyTm9kZSA9IHVuZGVmaW5lZDsKCmZ1bmN0aW9uIHNldHVw KCkKewogICAgc2xpZGVyQ2hhbmdlZCgpOwogICAgbXVzaWNOb2RlLmFkZEV2ZW50TGlzdGVuZXIo J3JhdGVjaGFuZ2UnLCByYXRlY2hhbmdlZCwgZmFsc2UpOwogICAgbXVzaWNOb2RlLmFkZEV2ZW50 TGlzdGVuZXIoJ3BhdXNlJywgc2xpZGVyQ2hhbmdlZCwgZmFsc2UpOwogICAgbXVzaWNOb2RlLmFk ZEV2ZW50TGlzdGVuZXIoJ3BsYXknLCBzbGlkZXJDaGFuZ2VkLCBmYWxzZSk7CiAgICBtdXNpY05v ZGUuYWRkRXZlbnRMaXN0ZW5lcignZW5kZWQnLCBzbGlkZXJDaGFuZ2VkLCBmYWxzZSk7Cn0KCmZ1 bmN0aW9uIHNsaWRlckNoYW5nZWQoKQp7Cgljb25zb2xlLmxvZygiVmFsdWU6ICIgKyBzbGlkZXJO b2RlLnZhbHVlKTsKCWNoYW5nZVJhdGUoc2xpZGVyTm9kZS52YWx1ZSkKfQoKZnVuY3Rpb24gY2hh bmdlUmF0ZShyYXRlKQp7CgltdXNpY05vZGUucGxheWJhY2tSYXRlID0gcmF0ZTsKCXJhdGVOb2Rl LmlubmVyVGV4dCA9ICJQbGF5YmFjayByYXRlOiAiICsgcmF0ZSArICJcblBhdXNlZDogIiArIG11 c2ljTm9kZS5wYXVzZWQgKyAiXG5FbmRlZDogIiArIG11c2ljTm9kZS5lbmRlZDsKfQoKZnVuY3Rp b24gcmF0ZWNoYW5nZWQoZXZ0KQp7CiAgICBpZiAocGFyc2VJbnQoc2xpZGVyTm9kZS52YWx1ZSku dG9GaXhlZCgyKSAhPSBtdXNpY05vZGUucGxheWJhY2tSYXRlLnRvRml4ZWQoMikpCiAgICAJbXVz aWNOb2RlLnBsYXliYWNrUmF0ZSA9IHNsaWRlck5vZGUudmFsdWU7Cn0KCjwvc2NyaXB0PgoKPGJv ZHkgb25sb2FkPSJzZXR1cCgpOyI+CjxhdWRpbyBpZD0ibXVzaWMiIG9ucGF1c2U9Im9uUGF1c2VD YWxsZWQoKTsiIGNvbnRyb2xzIHNyYz0iRmFpcnkgUHJpbmNlc3MubXAzIj48L2F1ZGlvPjxicj48 YnI+CjxkaXYgaWQ9InJhdGUiPlBsYXliYWNrIHJhdGU6IDE8L2Rpdj48YnI+CjxpbnB1dCBpZD0i cmFuZ2VDaGFuZ2VyIiB0eXBlPSJyYW5nZSIgbWluPSItMTYuMCIgbWF4PSIxNi4wIiB2YWx1ZT0i MSIgb25jaGFuZ2U9InNsaWRlckNoYW5nZWQoKTsiPgoKPC9ib2R5PgoKCjxzY3JpcHQ+Cm11c2lj Tm9kZSA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJtdXNpYyIpOwpyYXRlTm9kZSA9IGRvY3Vt ZW50LmdldEVsZW1lbnRCeUlkKCJyYXRlIik7CnNsaWRlck5vZGUgPSBkb2N1bWVudC5nZXRFbGVt ZW50QnlJZCgicmFuZ2VDaGFuZ2VyIik7Cjwvc2NyaXB0Pgo8L2h0bWw+Cg== 31421 2009-06-17 10:56:19 -0700 2009-06-17 11:00:03 -0700 proposed patch patch.txt text/plain 1950 eric.carlson SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA0NDc2OCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTcgQEAKKzIwMDktMDYtMTcgIEVyaWMgQ2FybHNvbiAgPGVyaWMuY2FybHNvbkBh cHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI2NDgyCisgICAgICAg IDxyZGFyOi8vcHJvYmxlbS82OTc4NTkwPiBXaGVuIHNldHRpbmcgcGxheWJhY2sgcmF0ZSB0byAw LCB0aGUgYXVkaW8gZWxlbWVudCBzdG9wcyAKKyAgICAgICAgcGxheWluZyAoZG9lc24ndCByZXN1 bWUgbGF0ZXIpCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9tYWMvTWVkaWFQbGF5ZXJQ cml2YXRlUVRLaXQubW06CisgICAgICAgIChXZWJDb3JlOjpNZWRpYVBsYXllclByaXZhdGU6OnNl dFJhdGUpOiBBbHdheXMgc2V0IHRoZSByYXRlLCBldmVuIHdoZW4gInBhdXNlZCIuCisKKyAgICAg ICAgKiBwbGF0Zm9ybS9ncmFwaGljcy93aW4vTWVkaWFQbGF5ZXJQcml2YXRlUXVpY2tUaW1lV2lu LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6Ok1lZGlhUGxheWVyUHJpdmF0ZTo6c2V0UmF0ZSk6IERp dHRvLgorCiAyMDA5LTA2LTE3ICBBZGFtIFJvYmVuICA8YXJvYmVuQGFwcGxlLmNvbT4KIAogICAg ICAgICBBZGQgYWxsIHRoZSBnZW5lcmF0ZWQgSlMqLmNwcCBmaWxlcyBiYWNrIHRvIFdlYkNvcmUu dmNwcm9qCkluZGV4OiBXZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL21hYy9NZWRpYVBsYXllclBy aXZhdGVRVEtpdC5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL21h Yy9NZWRpYVBsYXllclByaXZhdGVRVEtpdC5tbQkocmV2aXNpb24gNDQ3MjIpCisrKyBXZWJDb3Jl L3BsYXRmb3JtL2dyYXBoaWNzL21hYy9NZWRpYVBsYXllclByaXZhdGVRVEtpdC5tbQkod29ya2lu ZyBjb3B5KQpAQCAtNjA3LDggKzYwNyw3IEBAIHZvaWQgTWVkaWFQbGF5ZXJQcml2YXRlOjpzZXRS YXRlKGZsb2F0IHIKIHsKICAgICBpZiAoIW1ldGFEYXRhQXZhaWxhYmxlKCkpCiAgICAgICAgIHJl dHVybjsKLSAgICBpZiAoIXBhdXNlZCgpKQotICAgICAgICBbbV9xdE1vdmllLmdldCgpIHNldFJh dGU6cmF0ZV07CisgICAgW21fcXRNb3ZpZS5nZXQoKSBzZXRSYXRlOnJhdGVdOwogfQogCiBpbnQg TWVkaWFQbGF5ZXJQcml2YXRlOjpkYXRhUmF0ZSgpIGNvbnN0CkluZGV4OiBXZWJDb3JlL3BsYXRm b3JtL2dyYXBoaWNzL3dpbi9NZWRpYVBsYXllclByaXZhdGVRdWlja1RpbWVXaW4uY3BwCj09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT0KLS0tIFdlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3Mvd2luL01lZGlhUGxheWVyUHJpdmF0 ZVF1aWNrVGltZVdpbi5jcHAJKHJldmlzaW9uIDQ0NzIyKQorKysgV2ViQ29yZS9wbGF0Zm9ybS9n cmFwaGljcy93aW4vTWVkaWFQbGF5ZXJQcml2YXRlUXVpY2tUaW1lV2luLmNwcAkod29ya2luZyBj b3B5KQpAQCAtMjUyLDggKzI1Miw3IEBAIHZvaWQgTWVkaWFQbGF5ZXJQcml2YXRlOjpzZXRSYXRl KGZsb2F0IHIKIHsKICAgICBpZiAoIW1fcXRNb3ZpZSkKICAgICAgICAgcmV0dXJuOwotICAgIGlm ICghcGF1c2VkKCkpCi0gICAgICAgIG1fcXRNb3ZpZS0+c2V0UmF0ZShyYXRlKTsKKyAgICBtX3F0 TW92aWUtPnNldFJhdGUocmF0ZSk7CiB9CiAKIGludCBNZWRpYVBsYXllclByaXZhdGU6OmRhdGFS YXRlKCkgY29uc3QK