191259 2018-11-05 06:06:56 -0800 [GStreamer] video loop sometimes fails 2020-05-05 02:11:21 -0700 1 1 1 Unclassified WebKit Platform WebKit Nightly Build Unspecified Unspecified RESOLVED DUPLICATE 211182 P2 Normal --- 1 pnormand pnormand calvaris oldest_to_newest 1475050 0 pnormand 2018-11-05 06:06:56 -0800 I think this happens since bug 188645 was fixed. At EOS for a looping video the reported position and duration differ, so the media element doesn't seek back to the beginning... 1475053 1 353855 pnormand 2018-11-05 06:15:49 -0800 Created attachment 353855 Patch 1475431 2 353855 calvaris 2018-11-05 23:27:52 -0800 Comment on attachment 353855 Patch Patch is ok, but we should have a regression test for it. 1475442 3 pnormand 2018-11-06 00:55:36 -0800 (In reply to Xabier Rodríguez Calvar from comment #2) > Comment on attachment 353855 [details] > Patch > > Patch is ok, but we should have a regression test for it. Of course it should... But due to the inconsistent (flaky) nature of the issue, it's going to be hard to get one. 1475927 4 pnormand 2018-11-07 02:10:29 -0800 There's already a test BTW, marked flaky for a long time already, media/video-loop.html 1475951 5 calvaris 2018-11-07 03:51:13 -0800 (In reply to Philippe Normand from comment #4) > There's already a test BTW, marked flaky for a long time already, > media/video-loop.html Does it pass now reliably? 1475981 6 pnormand 2018-11-07 07:01:54 -0800 Even before the patch it passes reliably here. 1476822 7 calvaris 2018-11-08 23:52:52 -0800 (In reply to Philippe Normand from comment #6) > Even before the patch it passes reliably here. This is even weirder. This is an important feature that is broken, we should really find a way to test this and avoid regressions. Might be related to the video we are using? 1649131 8 pnormand 2020-05-05 02:11:21 -0700 *** This bug has been marked as a duplicate of bug 211182 *** 353855 2018-11-05 06:15:49 -0800 2020-05-05 02:11:10 -0700 Patch bug-191259-20181105141548.patch text/plain 3205 pnormand U3VidmVyc2lvbiBSZXZpc2lvbjogMjM3Nzk2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNzNlYTA4M2ZhZmU2MzU1 ODlmNzAwNzhhMWNlMzhkN2YzYmNjMDQ1Ny4uOTBhYmY0MDNhMTk5MzIxNjQwNmQyZDRkMDJjMmU0 MGJmMzg4OWViYyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDI0IEBACisyMDE4LTExLTA1ICBQaGls aXBwZSBOb3JtYW5kICA8cG5vcm1hbmRAaWdhbGlhLmNvbT4KKworICAgICAgICBbR1N0cmVhbWVy XSB2aWRlbyBsb29wIHNvbWV0aW1lcyBmYWlscworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0 Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTkxMjU5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgVGhpcyBjaGFuZ2UgZW5zdXJlcyB0aGUgbWVkaWEgcGxheWVy IHJlcG9ydHMgdmFsaWQgcGxheWJhY2sKKyAgICAgICAgcG9zaXRpb25zIGF0IEVPUy4gSWYgcGxh eWJhY2sgaXMgcnVubmluZyBiYWNrd2FyZHMgdGhlbiAwIGlzCisgICAgICAgIHJlcG9ydGVkLCBv dGhlcndpc2UgdGhlIHZhbHVlIG9mIHRoZSBkdXJhdGlvbiBpcyByZXBvcnRlZC4gVGhpcyBpcwor ICAgICAgICBkb25lIGJlZm9yZSBjaGVja2luZyB0aGUgY2FjaGVkIHZhbHVlIGJlY2F1c2UgYXQg RU9TIGl0IG1pZ2h0CisgICAgICAgIHNsaWdobHkgZGlmZmVyIGZyb20gdGhlIGR1cmF0aW9uIChm b3IgZm9yd2FyZCBwbGF5YmFjaykuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9nc3Ry ZWFtZXIvTWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6 Ok1lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lcjo6cGxheWJhY2tQb3NpdGlvbiBjb25zdCk6Cisg ICAgICAgIEFkZCAyIG1vcmUgZWFybHktcmV0dXJuIEVPUyBjYXNlcywgZGVwZW5kaW5nIG9uIHBs YXliYWNrIHJhdGUgd2UKKyAgICAgICAgY2FuIGVpdGhlciByZXR1cm4gMCBvciB0aGUgZHVyYXRp b24gdGhlcmUuCisgICAgICAgIChXZWJDb3JlOjpNZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXI6 OmRpZEVuZCk6IFVwZGF0ZSBkdXJhdGlvbiBhdAorICAgICAgICBFT1MgaW4gYWxsIGNhc2VzLCBl dmVuIHdoZW4gbm90IGxvb3BpbmcuIFRoaXMgaXMgdXNlZnVsIGZvciB0aGUKKyAgICAgICAgcGxh eWJhY2sgcG9zaXRpb24gZWFybHktcmV0dXJuIHZhbHVlLgorCiAyMDE4LTExLTA1ICBQaGlsaXBw ZSBOb3JtYW5kICA8cG5vcm1hbmRAaWdhbGlhLmNvbT4KIAogICAgICAgICBbR1N0cmVhbWVyXSBN b3ZlIGVsZW1lbnRzIHJlZ2lzdHJhdGlvbiB0byBHU3RyZWFtZXJDb21tb24KZGlmZiAtLWdpdCBh L1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2dzdHJlYW1lci9NZWRpYVBsYXllclBy aXZhdGVHU3RyZWFtZXIuY3BwIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0 cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lci5jcHAKaW5kZXggMDc0MDk2ZDAxODk5 ZjhmMDkxYWYyMWQ0YTNiNzdiNThmMjkwMzU3NC4uZjVmZWI2YzBlYTllMDdkZTczOWFkOTk5Njhk N2I4ODg2YTc3NWY4MyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhp Y3MvZ3N0cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lci5jcHAKKysrIGIvU291cmNl L1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdT dHJlYW1lci5jcHAKQEAgLTMzOSw4ICszMzksMTEgQEAgdm9pZCBNZWRpYVBsYXllclByaXZhdGVH U3RyZWFtZXI6OmNvbW1pdExvYWQoKQogCiBNZWRpYVRpbWUgTWVkaWFQbGF5ZXJQcml2YXRlR1N0 cmVhbWVyOjpwbGF5YmFja1Bvc2l0aW9uKCkgY29uc3QKIHsKLSAgICBpZiAobV9pc0VuZFJlYWNo ZWQgJiYgbV9zZWVraW5nKQotICAgICAgICByZXR1cm4gbV9zZWVrVGltZTsKKyAgICBpZiAobV9p c0VuZFJlYWNoZWQpIHsKKyAgICAgICAgaWYgKG1fc2Vla2luZykKKyAgICAgICAgICAgIHJldHVy biBtX3NlZWtUaW1lOworICAgICAgICByZXR1cm4gbV9wbGF5YmFja1JhdGUgPiAwID8gbV9kdXJh dGlvbkF0RU9TIDogTWVkaWFUaW1lOjp6ZXJvVGltZSgpOworICAgIH0KIAogICAgIC8vIFRoaXMg Y29uc3RhbnQgc2hvdWxkIHJlbWFpbiBsb3dlciB0aGFuIEhUTUxNZWRpYUVsZW1lbnQncyBtYXhU aW1ldXBkYXRlRXZlbnRGcmVxdWVuY3kuCiAgICAgc3RhdGljIGNvbnN0IFNlY29uZHMgcG9zaXRp b25DYWNoZVRocmVzaG9sZCA9IDIwMF9tczsKQEAgLTIxNTUsNyArMjE1OCw3IEBAIHZvaWQgTWVk aWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyOjp0aW1lQ2hhbmdlZCgpCiAKIHZvaWQgTWVkaWFQbGF5 ZXJQcml2YXRlR1N0cmVhbWVyOjpkaWRFbmQoKQogewotICAgIEdTVF9JTkZPKCJQbGF5YmFjayBl bmRlZCIpOworICAgIEdTVF9JTkZPX09CSkVDVChwaXBlbGluZSgpLCAiUGxheWJhY2sgZW5kZWQi KTsKIAogICAgIC8vIFN5bmNocm9uaXplIHBvc2l0aW9uIGFuZCBkdXJhdGlvbiB2YWx1ZXMgdG8g bm90IGNvbmZ1c2UgdGhlCiAgICAgLy8gSFRNTE1lZGlhRWxlbWVudC4gSW4gc29tZSBjYXNlcyBs aWtlIHJldmVyc2UgcGxheWJhY2sgdGhlCkBAIC0yMTY2LDE0ICsyMTY5LDE1IEBAIHZvaWQgTWVk aWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyOjpkaWRFbmQoKQogICAgICAgICBtX3BsYXllci0+ZHVy YXRpb25DaGFuZ2VkKCk7CiAKICAgICBtX2lzRW5kUmVhY2hlZCA9IHRydWU7Ci0gICAgdGltZUNo YW5nZWQoKTsKKyAgICBtX2R1cmF0aW9uQXRFT1MgPSBkdXJhdGlvbk1lZGlhVGltZSgpOwogCiAg ICAgaWYgKCFtX3BsYXllci0+Y2xpZW50KCkubWVkaWFQbGF5ZXJJc0xvb3BpbmcoKSkgewogICAg ICAgICBtX3BhdXNlZCA9IHRydWU7Ci0gICAgICAgIG1fZHVyYXRpb25BdEVPUyA9IGR1cmF0aW9u TWVkaWFUaW1lKCk7CiAgICAgICAgIGNoYW5nZVBpcGVsaW5lU3RhdGUoR1NUX1NUQVRFX1JFQURZ KTsKICAgICAgICAgbV9kb3dubG9hZEZpbmlzaGVkID0gZmFsc2U7CiAgICAgfQorCisgICAgdGlt ZUNoYW5nZWQoKTsKIH0KIAogdm9pZCBNZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXI6OmR1cmF0 aW9uQ2hhbmdlZCgpCg==