200341 2019-08-01 04:00:30 -0700 [GTK][WPE] Inconsistent reporting of current URI in WebKitWebView:load-changed signal 2019-09-03 03:26:53 -0700 1 1 1 Unclassified WebKit WebKit API Other Unspecified Unspecified RESOLVED DUPLICATE 201176 P2 Normal --- 1 aperez webkit-unassigned bugs-noreply cgarcia csaavedra mcatanzaro oldest_to_newest 1557569 0 375303 aperez 2019-08-01 04:00:30 -0700 Created attachment 375303 Test program, triggers a second load 200ms after the first is committed If a new load is triggered soon after a previous one has reached the WEBKIT_LOAD_COMMITTED state, the next notified WEBKIT_LOAD_STARTED will have the old URL most of the time (9 out of 10 times, at least) instead of the new one being loaded. 1557570 1 aperez 2019-08-01 04:01:28 -0700 Typical output from the “load-test.c” program (attached): % ./test-load ** Message: 13:57:12.909: Load started <http://apple.com/> ** Message: 13:57:13.443: Load redirected <https://www.apple.com/> ** Message: 13:57:13.810: Load committed <https://www.apple.com/> ** Message: 13:57:14.011: on_load_started_timeout: triggering load for igalia.com ** Message: 13:57:15.135: on_web_view_load_failed: event=committed error=Load request cancelled uri=https://www.apple.com/ web_view_uri=https://www.apple.com/ ** Message: 13:57:15.135: Load finished <https://www.apple.com/> ** Message: 13:57:15.135: Load started <https://www.apple.com/> ** Message: 13:57:15.558: Load redirected <https://www.igalia.com/> ** Message: 13:57:15.842: Load committed <https://www.igalia.com/> ** Message: 13:57:18.322: Load finished <https://www.igalia.com/> 1557571 2 aperez 2019-08-01 04:05:05 -0700 A couple of times I have also seen the new URL being used for the WEBKIT_LOAD_FINISHED of the previous page load. I don't seem to be able of getting that today (it's much less frequent), but I have this log saved from yesterday: ** Message: 18:39:03.581: Load started <http://apple.com/> ** Message: 18:39:03.587: Load redirected <https://www.apple.com/> ** Message: 18:39:04.395: Load committed <https://www.apple.com/> ** Message: 18:39:04.596: on_load_started_timeout: triggering load for igalia.com ** Message: 18:39:04.599: on_web_view_load_failed: event=committed error=Load request cancelled ** Message: 18:39:04.599: Load finished <https://igalia.com/> ** Message: 18:39:04.599: Load started <https://igalia.com/> ** Message: 18:39:05.042: Load redirected <https://www.igalia.com/> ** Message: 18:39:05.297: Load committed <https://www.igalia.com/> 1566175 3 cgarcia 2019-08-29 03:34:02 -0700 I think this has been fixed by r249193. Could you confirm? 1567322 4 aperez 2019-09-03 03:26:13 -0700 (In reply to Carlos Garcia Campos from comment #3) > I think this has been fixed by r249193. Could you confirm? Yes, it has been fixed for the test program attached to the bug, and I can confirm that it also fixes the issue for the bigger program where I noticed this before. \o/ 1567323 5 aperez 2019-09-03 03:26:53 -0700 *** This bug has been marked as a duplicate of bug 201176 *** 375303 2019-08-01 04:00:30 -0700 2019-08-01 04:00:30 -0700 Test program, triggers a second load 200ms after the first is committed test-load.c text/x-csrc 4257 aperez LyoKICogQnVpbGQgd2l0aDoKICogICAkQ0MgYHBrZy1jb25maWcgd2Via2l0Mmd0ay00LjAgLS1j ZmxhZ3MgLS1saWJzYCAtbyB0ZXN0LWxvYWQgdGVzdC1sb2FkLmMKICovCgojaW5jbHVkZSA8d2Vi a2l0Mi93ZWJraXQyLmg+CgpzdGF0aWMgdm9pZApvbl9hcHBfYWN0aW9uX3F1aXQgKEdTaW1wbGVB Y3Rpb24gKmFjdGlvbiBHX0dOVUNfVU5VU0VELAogICAgICAgICAgICAgICAgICAgIEdWYXJpYW50 ICAgICAgKnBhcmFtIEdfR05VQ19VTlVTRUQsCiAgICAgICAgICAgICAgICAgICAgdm9pZCAgICAg ICAgICAqdXNlcl9kYXRhKQp7CiAgICBnX2FwcGxpY2F0aW9uX3F1aXQgKEdfQVBQTElDQVRJT04g KHVzZXJfZGF0YSkpOwp9CgpzdGF0aWMgZ2Jvb2xlYW4Kb25fbG9hZF9zdGFydGVkX3RpbWVvdXQg KFdlYktpdFdlYlZpZXcgKndlYl92aWV3KQp7CiAgICBnX21lc3NhZ2UgKCIlczogdHJpZ2dlcmlu ZyBsb2FkIGZvciBpZ2FsaWEuY29tIiwgX19mdW5jX18pOwogICAgd2Via2l0X3dlYl92aWV3X2xv YWRfdXJpICh3ZWJfdmlldywgImh0dHBzOi8vaWdhbGlhLmNvbSIpOwogICAgcmV0dXJuIEdfU09V UkNFX1JFTU9WRTsKfQoKc3RhdGljIGNvbnN0IGNoYXIqCmdlbnVtX25pY2sgKEdUeXBlIGVudW1f dHlwZSwgaW50IHZhbHVlKQp7CiAgICBHRW51bUNsYXNzICplbnVtX2NsYXNzID0gKEdFbnVtQ2xh c3MqKSBnX3R5cGVfY2xhc3NfcmVmIChlbnVtX3R5cGUpOwogICAgR0VudW1WYWx1ZSAqZW51bV92 YWx1ZSA9IGdfZW51bV9nZXRfdmFsdWUgKGVudW1fY2xhc3MsIHZhbHVlKTsKICAgIGNvbnN0IGNo YXIgKm5pY2sgPSBlbnVtX3ZhbHVlID8gZW51bV92YWx1ZS0+dmFsdWVfbmljayA6IE5VTEw7CiAg ICBnX3R5cGVfY2xhc3NfdW5yZWYgKGVudW1fY2xhc3MpOwogICAgcmV0dXJuIG5pY2s7Cn0KCnN0 YXRpYyB2b2lkCm9uX3dlYl92aWV3X2xvYWRfY2hhbmdlZCAoV2ViS2l0V2ViVmlldyAgICp3ZWJf dmlldywKICAgICAgICAgICAgICAgICAgICAgICAgICBXZWJLaXRMb2FkRXZlbnQgIGV2ZW50KQp7 CiAgICBzdGF0aWMgZ2Jvb2xlYW4gZmlyc3RfcnVuID0gVFJVRTsKCiAgICBzd2l0Y2ggKGV2ZW50 KSB7CiAgICAgICAgY2FzZSBXRUJLSVRfTE9BRF9TVEFSVEVEOgogICAgICAgICAgICBicmVhazsK ICAgICAgICBjYXNlIFdFQktJVF9MT0FEX0NPTU1JVFRFRDoKICAgICAgICAgICAgaWYgKGZpcnN0 X3J1bikgewogICAgICAgICAgICAgICAgZmlyc3RfcnVuID0gRkFMU0U7CiAgICAgICAgICAgICAg ICBnX3RpbWVvdXRfYWRkICgyMDAsIEdfU09VUkNFX0ZVTkMgKG9uX2xvYWRfc3RhcnRlZF90aW1l b3V0KSwgd2ViX3ZpZXcpOwogICAgICAgICAgICB9CiAgICAgICAgICAgIGJyZWFrOwogICAgICAg IGNhc2UgV0VCS0lUX0xPQURfUkVESVJFQ1RFRDoKICAgICAgICAgICAgYnJlYWs7CiAgICAgICAg Y2FzZSBXRUJLSVRfTE9BRF9GSU5JU0hFRDoKICAgICAgICAgICAgYnJlYWs7CiAgICB9CgogICAg Y29uc3QgY2hhciAqdXJpID0gd2Via2l0X3dlYl92aWV3X2dldF91cmkgKHdlYl92aWV3KSA/IDog Ii0iOwogICAgZ19tZXNzYWdlICgiTG9hZCAlcyA8JXM+IiwgZ2VudW1fbmljayAoV0VCS0lUX1RZ UEVfTE9BRF9FVkVOVCwgZXZlbnQpLCB1cmkpOwp9CgpzdGF0aWMgZ2Jvb2xlYW4Kb25fd2ViX3Zp ZXdfbG9hZF9mYWlsZWQgKFdlYktpdFdlYlZpZXcgICAqd2ViX3ZpZXcsCiAgICAgICAgICAgICAg ICAgICAgICAgICBXZWJLaXRMb2FkRXZlbnQgIGV2ZW50LAogICAgICAgICAgICAgICAgICAgICAg ICAgY2hhciAgICAgICAgICAgICp1cmksCiAgICAgICAgICAgICAgICAgICAgICAgICBHRXJyb3Ig ICAgICAgICAgKmVycm9yKQp7CiAgICBnX21lc3NhZ2UgKCIlczogZXZlbnQ9JXMgZXJyb3I9JXMg dXJpPSVzIHdlYl92aWV3X3VyaT0lcyIsCiAgICAgICAgICAgICAgIF9fZnVuY19fLAogICAgICAg ICAgICAgICBnZW51bV9uaWNrIChXRUJLSVRfVFlQRV9MT0FEX0VWRU5ULCBldmVudCksCiAgICAg ICAgICAgICAgIGVycm9yID8gZXJyb3ItPm1lc3NhZ2UgOiAiLSIsCiAgICAgICAgICAgICAgIHVy aSA/IDogIi0iLAogICAgICAgICAgICAgICB3ZWJraXRfd2ViX3ZpZXdfZ2V0X3VyaSAod2ViX3Zp ZXcpID8gOiAiLSIpOwoKICAgIGlmIChlcnJvciAmJiBnX2Vycm9yX21hdGNoZXMgKGVycm9yLCAg V0VCS0lUX1BMVUdJTl9FUlJPUiwgV0VCS0lUX1BMVUdJTl9FUlJPUl9XSUxMX0hBTkRMRV9MT0FE KSkgewogICAgICAgIGdfbWVzc2FnZSAoIiAgIHJlc291cmNlIHRvIGJlIHNob3duIGJ5IGEgcGx1 Zy1pbi9tZWRpYSBlbmdpbmUsIGxldCBXZWJLaXQgaGFuZGxlIGl0Iik7CiAgICAgICAgcmV0dXJu IEZBTFNFOwogICAgfQoKICAgIHJldHVybiBUUlVFOwp9CgpzdGF0aWMgZ2Jvb2xlYW4Kb25fd2Vi X3ZpZXdfbG9hZF9mYWlsZWRfd2l0aF90bHNfZXJyb3JzIChXZWJLaXRXZWJWaWV3ICAgICAgICAq d2ViX3ZpZXcsCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgY2hhciAg ICAgICAgICAgICAgICAgKnVyaSwKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICBHVGxzQ2VydGlmaWNhdGUgICAgICAqY2VydGlmaWNhdGUsCiAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgR1Rsc0NlcnRpZmljYXRlRmxhZ3MgIGVycm9ycykKewog ICAgZ19tZXNzYWdlICgiJXM6IHVyaT0lcyIsIF9fZnVuY19fLCB1cmkgPyA6ICItIik7CiAgICAv LyBEbyBub3QgZW1pdCA6OmxvYWQtZmFpbGVkLCBvbmx5IDo6bG9hZC1jaGFuZ2VkLgogICAgcmV0 dXJuIFRSVUU7Cn0KCnN0YXRpYyB2b2lkCm9uX2FwcF9zdGFydHVwIChHdGtBcHBsaWNhdGlvbiAq YXBwKQp7CiAgICBzdGF0aWMgY29uc3QgR0FjdGlvbkVudHJ5IGFwcF9hY3Rpb25zW10gPSB7CiAg ICAgICAgeyAubmFtZSA9ICJxdWl0IiwgLmFjdGl2YXRlID0gb25fYXBwX2FjdGlvbl9xdWl0IH0s CiAgICB9OwogICAgZ19hY3Rpb25fbWFwX2FkZF9hY3Rpb25fZW50cmllcyAoR19BQ1RJT05fTUFQ IChhcHApLCBhcHBfYWN0aW9ucywKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg IEdfTl9FTEVNRU5UUyAoYXBwX2FjdGlvbnMpLCBhcHApOwp9CgpzdGF0aWMgdm9pZApvbl9hcHBf YWN0aXZhdGUgKEd0a0FwcGxpY2F0aW9uICphcHApCnsKICAgIEd0a1dpZGdldCAqd2luZG93ID0g Z3RrX2FwcGxpY2F0aW9uX3dpbmRvd19uZXcgKGFwcCk7CiAgICBHdGtXaWRnZXQgKndlYl92aWV3 ID0gd2Via2l0X3dlYl92aWV3X25ldyAoKTsKCiAgICBndGtfYXBwbGljYXRpb25fd2luZG93X3Nl dF9zaG93X21lbnViYXIgKEdUS19BUFBMSUNBVElPTl9XSU5ET1cgKHdpbmRvdyksIEZBTFNFKTsK ICAgIGd0a19jb250YWluZXJfYWRkIChHVEtfQ09OVEFJTkVSICh3aW5kb3cpLCB3ZWJfdmlldyk7 CiAgICBndGtfd2lkZ2V0X3Nob3dfYWxsICh3aW5kb3cpOwoKICAgIGdfc2lnbmFsX2Nvbm5lY3Qg KHdlYl92aWV3LCAibG9hZC1jaGFuZ2VkIiwKICAgICAgICAgICAgICAgICAgICAgIEdfQ0FMTEJB Q0sgKG9uX3dlYl92aWV3X2xvYWRfY2hhbmdlZCksIE5VTEwpOwogICAgZ19zaWduYWxfY29ubmVj dCAod2ViX3ZpZXcsICJsb2FkLWZhaWxlZCIsCiAgICAgICAgICAgICAgICAgICAgICBHX0NBTExC QUNLIChvbl93ZWJfdmlld19sb2FkX2ZhaWxlZCksIE5VTEwpOwogICAgZ19zaWduYWxfY29ubmVj dCAod2ViX3ZpZXcsICJsb2FkLWZhaWxlZC13aXRoLXRscy1lcnJvcnMiLAogICAgICAgICAgICAg ICAgICAgICAgR19DQUxMQkFDSyAob25fd2ViX3ZpZXdfbG9hZF9mYWlsZWRfd2l0aF90bHNfZXJy b3JzKSwgTlVMTCk7CgogICAgd2Via2l0X3dlYl92aWV3X2xvYWRfdXJpIChXRUJLSVRfV0VCX1ZJ RVcgKHdlYl92aWV3KSwgImh0dHA6Ly9hcHBsZS5jb20iKTsKfQoKaW50Cm1haW4gKGludCBhcmdj LCBjaGFyICphcmd2W10pCnsKICAgIGdfYXV0b3B0cihHdGtBcHBsaWNhdGlvbikgYXBwID0gZ3Rr X2FwcGxpY2F0aW9uX25ldyAoTlVMTCwgR19BUFBMSUNBVElPTl9GTEFHU19OT05FKTsKICAgIGdf c2lnbmFsX2Nvbm5lY3QgKGFwcCwgImFjdGl2YXRlIiwgR19DQUxMQkFDSyAob25fYXBwX2FjdGl2 YXRlKSwgTlVMTCk7CiAgICBnX3NpZ25hbF9jb25uZWN0IChhcHAsICJzdGFydHVwIiwgR19DQUxM QkFDSyAob25fYXBwX3N0YXJ0dXApLCBOVUxMKTsKICAgIHJldHVybiBnX2FwcGxpY2F0aW9uX3J1 biAoR19BUFBMSUNBVElPTiAoYXBwKSwgYXJnYywgYXJndik7Cn0K