171722 2017-05-05 04:05:34 -0700 [GStreamer] Do not report more errors after the first one 2017-05-05 05:05:49 -0700 1 1 1 Unclassified WebKit Platform WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED Gtk, LayoutTestFailure P2 Normal --- 1 cgarcia webkit-unassigned bugs-noreply calvaris oldest_to_newest 1305056 0 cgarcia 2017-05-05 04:05:34 -0700 We can receive several error messages for the same error from different elements. That's confusing the media source selection algorithm implementation. I don't know if didn't happen with previous versions of GST, but since the upgrade to 1.10.4 several tests are failing because of this. 1305057 1 309158 cgarcia 2017-05-05 04:07:32 -0700 Created attachment 309158 Patch 1305074 2 309158 calvaris 2017-05-05 04:57:04 -0700 Comment on attachment 309158 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=309158&action=review > Source/WebCore/ChangeLog:8 > + We can receive several error messages for the same error from different elements. That's confusing the media ... That's confusing FOR the media source ... 1305079 3 cgarcia 2017-05-05 05:00:21 -0700 (In reply to Xabier Rodríguez Calvar from comment #2) > Comment on attachment 309158 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=309158&action=review > > > Source/WebCore/ChangeLog:8 > > + We can receive several error messages for the same error from different elements. That's confusing the media > > ... That's confusing FOR the media source ... I meant the media source alg impl gets confused by this. Not sure it's correct English, though. 1305081 4 cgarcia 2017-05-05 05:05:49 -0700 Committed r216240: <http://trac.webkit.org/changeset/216240> 309158 2017-05-05 04:07:32 -0700 2017-05-05 04:57:04 -0700 Patch wk-gst-multiple-errors.diff text/plain 2375 cgarcia ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCA2ZjU1MjAwYjhjNi4uYjY2OTZiNWYxNjIgMTAwNjQ0Ci0tLSBhL1NvdXJj ZS9XZWJDb3JlL0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEs NSArMSwyNSBAQAogMjAxNy0wNS0wNSAgQ2FybG9zIEdhcmNpYSBDYW1wb3MgIDxjZ2FyY2lhQGln YWxpYS5jb20+CiAKKyAgICAgICAgW0dTdHJlYW1lcl0gRG8gbm90IHJlcG9ydCBtb3JlIGVycm9y cyBhZnRlciB0aGUgZmlyc3Qgb25lCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No b3dfYnVnLmNnaT9pZD0xNzE3MjIKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMh KS4KKworICAgICAgICBXZSBjYW4gcmVjZWl2ZSBzZXZlcmFsIGVycm9yIG1lc3NhZ2VzIGZvciB0 aGUgc2FtZSBlcnJvciBmcm9tIGRpZmZlcmVudCBlbGVtZW50cy4gVGhhdCdzIGNvbmZ1c2luZyB0 aGUgbWVkaWEKKyAgICAgICAgc291cmNlIHNlbGVjdGlvbiBhbGdvcml0aG0gaW1wbGVtZW50YXRp b24uIEkgZG9uJ3Qga25vdyBpZiBkaWRuJ3QgaGFwcGVuIHdpdGggcHJldmlvdXMgdmVyc2lvbnMg b2YgR1NULCBidXQKKyAgICAgICAgc2luY2UgdGhlIHVwZ3JhZGUgdG8gMS4xMC40IHNldmVyYWwg dGVzdHMgYXJlIGZhaWxpbmcgYmVjYXVzZSBvZiB0aGlzLgorCisgICAgICAgIEZpeGVzOiBtZWRp YS92aWRlby1lcnJvci1kb2VzLW5vdC1leGlzdC5odG1sCisgICAgICAgICAgICAgICBtZWRpYS92 aWRlby1sb2FkLW5ldHdvcmtTdGF0ZS5odG1sCisgICAgICAgICAgICAgICBtZWRpYS92aWRlby1z b3VyY2UtZXJyb3IuaHRtbAorICAgICAgICAgICAgICAgbWVkaWEvdmlkZW8tc291cmNlLW5vbmUt c3VwcG9ydGVkLmh0bWwKKyAgICAgICAgICAgICAgIG1lZGlhL3ZpZGVvLXNvdXJjZS1tb3ZlZC5o dG1sCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvTWVkaWFQbGF5ZXJQ cml2YXRlR1N0cmVhbWVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6Ok1lZGlhUGxheWVyUHJpdmF0 ZUdTdHJlYW1lcjo6aGFuZGxlTWVzc2FnZSk6IFJldHVybiBlYXJseSBhbHNvIHdoZW4gYW4gZXJy b3IgYWxyZWFkeSBvY2N1cmVkLgorCisyMDE3LTA1LTA1ICBDYXJsb3MgR2FyY2lhIENhbXBvcyAg PGNnYXJjaWFAaWdhbGlhLmNvbT4KKwogICAgICAgICBbR1N0cmVhbWVyXSBGaXggaGFuZGxpbmcg b2YgZ3N0IGVycm9ycyBpbiBNZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXI6OmhhbmRsZU1lc3Nh Z2UKICAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE3MTcy MQogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFt ZXIvTWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsYXRm b3JtL2dyYXBoaWNzL2dzdHJlYW1lci9NZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXIuY3BwCmlu ZGV4IGY4M2ZhOTVjYTFlLi5iMzU2MDM4Y2RiNyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUv cGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lci5j cHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL01lZGlh UGxheWVyUHJpdmF0ZUdTdHJlYW1lci5jcHAKQEAgLTkzNyw5ICs5MzcsNyBAQCB2b2lkIE1lZGlh UGxheWVyUHJpdmF0ZUdTdHJlYW1lcjo6aGFuZGxlTWVzc2FnZShHc3RNZXNzYWdlKiBtZXNzYWdl KQogICAgIEdTVF9ERUJVRygiTWVzc2FnZSAlcyByZWNlaXZlZCBmcm9tIGVsZW1lbnQgJXMiLCBH U1RfTUVTU0FHRV9UWVBFX05BTUUobWVzc2FnZSksIEdTVF9NRVNTQUdFX1NSQ19OQU1FKG1lc3Nh Z2UpKTsKICAgICBzd2l0Y2ggKEdTVF9NRVNTQUdFX1RZUEUobWVzc2FnZSkpIHsKICAgICBjYXNl IEdTVF9NRVNTQUdFX0VSUk9SOgotICAgICAgICBpZiAobV9yZXNldFBpcGVsaW5lKQotICAgICAg ICAgICAgYnJlYWs7Ci0gICAgICAgIGlmIChtX21pc3NpbmdQbHVnaW5zQ2FsbGJhY2spCisgICAg ICAgIGlmIChtX3Jlc2V0UGlwZWxpbmUgfHwgbV9taXNzaW5nUGx1Z2luc0NhbGxiYWNrIHx8IG1f ZXJyb3JPY2N1cmVkKQogICAgICAgICAgICAgYnJlYWs7CiAgICAgICAgIGdzdF9tZXNzYWdlX3Bh cnNlX2Vycm9yKG1lc3NhZ2UsICZlcnIub3V0UHRyKCksICZkZWJ1Zy5vdXRQdHIoKSk7CiAgICAg ICAgIEdTVF9FUlJPUigiRXJyb3IgJWQ6ICVzICh1cmw9JXMpIiwgZXJyLT5jb2RlLCBlcnItPm1l c3NhZ2UsIG1fdXJsLnN0cmluZygpLnV0ZjgoKS5kYXRhKCkpOwo=