36007 2010-03-11 03:13:03 -0800 [GStreamer] progressTimer never stopped after EOS 2010-03-15 01:20:27 -0700 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) PC OS X 10.5 RESOLVED FIXED P2 Normal --- 1 pnormand webkit-unassigned oldest_to_newest 198537 0 pnormand 2010-03-11 03:13:03 -0800 The HTMLMediaElement progressTimer is not stopped after media playback ended. I think this is because we pause the pipeline in our didEnd() method, so the media element thinks the playback is paused, the timer never gets a chance to stop. 198538 1 pnormand 2010-03-11 03:14:28 -0800 It is easy to check with the launcher, activate Media logging with WEBKIT_DEBUG, watch a HTML5 video on youtube, wait the end and observe the progress timer still running, forever. 198561 2 50489 pnormand 2010-03-11 05:19:34 -0800 Created attachment 50489 don't mark the pipeline as paused on EOS 198786 3 50489 gustavo 2010-03-11 14:39:55 -0800 Comment on attachment 50489 don't mark the pipeline as paused on EOS Hrm. How about we enable the play/pause test somehow? That would rock =(. Sounds good to me for now, though. 199572 4 pnormand 2010-03-15 01:20:27 -0700 Landed in r55987. 50489 2010-03-11 05:19:34 -0800 2010-03-11 14:39:54 -0800 don't mark the pipeline as paused on EOS dont-mark-the-pipeline-as-paused-on-EOS.patch text/plain 2237 pnormand RnJvbSA3YzA2YTY5ZGY3ZTc5OWRjNDdkZjI1YjJjNjY1ZWNmNGMxZDU5MjEwIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBQaGlsaXBwZSBOb3JtYW5kIDxwbm9ybWFuZEBpZ2FsaWEuY29t PgpEYXRlOiBUaHUsIDExIE1hciAyMDEwIDEyOjE1OjEzICswMTAwClN1YmplY3Q6IFtQQVRDSF0g ZG9uJ3QgbWFyayB0aGUgcGlwZWxpbmUgYXMgcGF1c2VkIG9uIEVPUwoKLS0tCiBXZWJDb3JlL0No YW5nZUxvZyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICB8ICAgMTYgKysrKysrKysr KysrKysrKwogLi4uL2dzdHJlYW1lci9NZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXIuY3BwICAg ICAgfCAgICAzICsrLQogMiBmaWxlcyBjaGFuZ2VkLCAxOCBpbnNlcnRpb25zKCspLCAxIGRlbGV0 aW9ucygtKQoKZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VM b2cKaW5kZXggYjgyMjYzOS4uMDYyM2NkOCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cK KysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxOSBAQAorMjAxMC0wMy0xMSAgUGhp bGlwcGUgTm9ybWFuZCAgPHBub3JtYW5kQGlnYWxpYS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQg YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW0dTdHJlYW1lcl0gcHJvZ3Jlc3NUaW1lciBu ZXZlciBzdG9wcGVkIGFmdGVyIEVPUworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z aG93X2J1Zy5jZ2k/aWQ9MzYwMDcKKworICAgICAgICBNYXJrIHRoZSBwaXBlbGluZSBhcyBwYXVz ZWQgb25seSBpZiBwbGF5YmFjayBoYXNuJ3QgZW5kZWQKKyAgICAgICAgeWV0LiBXaGVuIGRpZEVu ZCgpIGlzIHRyaWdnZXJlZCwgdGhlIHBpcGVsaW5lIGlzIHBhdXNlZCAoYnV0CisgICAgICAgIHBh dXNlZCgpIHN0aWxsIHJldHVybnMgZmFsc2UpIGFuZCBhIHRpbWV1cGRhdGUgZXZlbnQgaXMgZmly ZWQsCisgICAgICAgIHRob3NlIDIgY29uZGl0aW9ucyBhbGxvdyB0aGUgSFRNTE1lZGlhRWxlbWVu dCB0byBzdG9wIHRoZQorICAgICAgICBwcm9ncmVzc1RpbWVyIChpbiB1cGRhdGVQbGF5ZXJTdGF0 ZSwgdmlhIG1lZGlhUGxheWVyVGltZUNoYW5nZWQpLgorCisgICAgICAgICogcGxhdGZvcm0vZ3Jh cGhpY3MvZ3N0cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lci5jcHA6CisgICAgICAg IChXZWJDb3JlOjpNZWRpYVBsYXllclByaXZhdGU6OnVwZGF0ZVN0YXRlcyk6CisKIDIwMTAtMDMt MTEgIENzYWJhIE9zenRyb2dvbsOhYyAgPG9zc3lAd2Via2l0Lm9yZz4KIAogICAgICAgICBVbnJl dmlld2VkIGJ1aWxkZml4IGFmdGVyIHI1NTgyMy4gKFRvIGZpeCBRdCAtLW1pbmltYWwgYnVpbGQu KQpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvTWVkaWFQ bGF5ZXJQcml2YXRlR1N0cmVhbWVyLmNwcCBiL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0 cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lci5jcHAKaW5kZXggMmFjZjE0OS4uZDA3 ZTQzYSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvTWVk aWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyLmNwcAorKysgYi9XZWJDb3JlL3BsYXRmb3JtL2dyYXBo aWNzL2dzdHJlYW1lci9NZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXIuY3BwCkBAIC04NzUsOCAr ODc1LDkgQEAgdm9pZCBNZWRpYVBsYXllclByaXZhdGU6OnVwZGF0ZVN0YXRlcygpCiAgICAgICAg ICAgICAgICAgICAgIExPR19WRVJCT1NFKE1lZGlhLCAiW0J1ZmZlcmluZ10gUmVzdGFydGluZyBw bGF5YmFjay4iKTsKICAgICAgICAgICAgICAgICAgICAgZ3N0X2VsZW1lbnRfc2V0X3N0YXRlKG1f cGxheUJpbiwgR1NUX1NUQVRFX1BMQVlJTkcpOwogICAgICAgICAgICAgICAgIH0KLSAgICAgICAg ICAgIH0gZWxzZSBpZiAoIW1fYnVmZmVyaW5nKQorICAgICAgICAgICAgfSBlbHNlIGlmICghbV9i dWZmZXJpbmcgJiYgKGN1cnJlbnRUaW1lKCkgPCBkdXJhdGlvbigpKSkgewogICAgICAgICAgICAg ICAgIG1fcGF1c2VkID0gdHJ1ZTsKKyAgICAgICAgICAgIH0KICAgICAgICAgfSBlbHNlIGlmIChz dGF0ZSA9PSBHU1RfU1RBVEVfUExBWUlORykgewogICAgICAgICAgICAgbV9yZWFkeVN0YXRlID0g TWVkaWFQbGF5ZXI6OkhhdmVFbm91Z2hEYXRhOwogICAgICAgICAgICAgbV9wYXVzZWQgPSBmYWxz ZTsKLS0gCjEuNi4zLjM=