11429 2006-10-27 04:33:19 -0700 Startup crash on Qt platform 2006-10-31 10:33:00 -0800 1 1 1 Unclassified WebKit Platform 420+ PC Linux RESOLVED FIXED P2 Normal --- 1 zack webkit-unassigned oldest_to_newest 49380 0 zack 2006-10-27 04:33:19 -0700 When calling FrameQtClient::openUrl for the first time, the owned frames doesn't have a document set on it at all. In turn ResourceLoader depends on the DocumentLoader in order to work and Qt's ResourceLoaderManager depends on it being present in order to set Frame (in order to call checkLoaded). openUrl used to let through creation of ResourceLoader with no parent document which causes crashes due to above described dependencies. The patch explictly creates an empty document if one isn't already present. 49363 1 11237 zack 2006-10-27 04:34:41 -0700 Created attachment 11237 Mentioned patch to fix the crash. 48573 2 11288 zimmermann 2006-10-30 11:58:26 -0800 Created attachment 11288 Updated patch with ChangeLog 48574 3 11288 andersca 2006-10-30 11:59:05 -0800 Comment on attachment 11288 Updated patch with ChangeLog r=me 48575 4 11288 andersca 2006-10-30 11:59:15 -0800 Comment on attachment 11288 Updated patch with ChangeLog r=me 48222 5 ap 2006-10-31 10:33:00 -0800 This was landed by Anders in r17441. 11237 2006-10-27 04:34:41 -0700 2006-10-30 11:58:26 -0800 Mentioned patch to fix the crash. resclient.diff text/plain 920 zack SW5kZXg6IFdlYkNvcmUvcGxhdGZvcm0vcXQvRnJhbWVRdENsaWVudC5jcHAKPT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot LS0gV2ViQ29yZS9wbGF0Zm9ybS9xdC9GcmFtZVF0Q2xpZW50LmNwcCAgICAgICAocmV2aXNpb24g MTczNjYpCisrKyBXZWJDb3JlL3BsYXRmb3JtL3F0L0ZyYW1lUXRDbGllbnQuY3BwICAgICAgICh3 b3JraW5nIGNvcHkpCkBAIC02NiwyICs2NiwyIEBAIHZvaWQgRnJhbWVRdENsaWVudERlZmF1bHQ6 Om9wZW5VUkwoY29uc3QKICAgICBtX2ZyYW1lLT5kaWRPcGVuVVJMKHVybCk7CiAgICAgbV9hc3Np Z25lZE1pbWV0eXBlID0gZmFsc2U7CgorICAgIGlmICghbV9mcmFtZS0+ZG9jdW1lbnQoKSkKKyAg ICAgICAgbV9mcmFtZS0+Y3JlYXRlRW1wdHlEb2N1bWVudCgpOworICAgIEFTU0VSVChtX2ZyYW1l LT5kb2N1bWVudCgpKTsKICAgICBSZXNvdXJjZVJlcXVlc3QgcmVxdWVzdCh1cmwpOwotICAgIFJl ZlB0cjxSZXNvdXJjZUxvYWRlcj4gbG9hZGVyID0gUmVzb3VyY2VMb2FkZXI6OmNyZWF0ZShyZXF1 ZXN0LCB0aGlzLCBtX2ZyYW1lLT5kb2N1bWVudCgpID8gbV9mcmFtZS0+ZG9jdW1lbnQoKS0+ZG9j TG9hZGVyKCkgOiAwKTsKKyAgICBSZWZQdHI8UmVzb3VyY2VMb2FkZXI+IGxvYWRlciA9IFJlc291 cmNlTG9hZGVyOjpjcmVhdGUocmVxdWVzdCwgdGhpcywKKyAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgbV9mcmFtZS0+ZG9jdW1lbnQoKS0+ ZG9jTG9hZGVyKCkpOwogfQoKIHZvaWQgRnJhbWVRdENsaWVudERlZmF1bHQ6OnN1Ym1pdEZvcm0o Y29uc3QgU3RyaW5nJiBtZXRob2QsIGNvbnN0IEtVUkwmIHVybCwgY29uc3QgRm9ybURhdGEqIHBv c3REYXRhKQo= 11288 2006-10-30 11:58:26 -0800 2006-10-30 11:59:15 -0800 Updated patch with ChangeLog crash-fix.diff text/plain 1437 zimmermann SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShSZXZpc2lvbiAxNzQ0MCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCShBcmJlaXRza29waWUpCkBA IC0xLDMgKzEsMTUgQEAKKzIwMDYtMTAtMzAgIE5pa29sYXMgWmltbWVybWFubiAgPHppbW1lcm1h bm5Aa2RlLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAg ICAgICBQYXRjaCBieSBaYWNrIHJ1c2luIHRvIGZpeDogCisgICAgICAgIGh0dHA6Ly9idWdzLndl YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTExNDI5CisKKyAgICAgICAgRml4IHN0YXJ0dXAgY3Jh c2guCisKKyAgICAgICAgKiBwbGF0Zm9ybS9xdC9GcmFtZVF0Q2xpZW50LmNwcDoKKyAgICAgICAg KFdlYkNvcmU6OkZyYW1lUXRDbGllbnREZWZhdWx0OjpvcGVuVVJMKToKKwogMjAwNi0xMC0zMCAg WmFjaCBSdXNpbiAgPHphY2tAa2RlLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBhbmQgbGFuZGVk IGJ5IEFuZGVycy4KSW5kZXg6IFdlYkNvcmUvcGxhdGZvcm0vcXQvRnJhbWVRdENsaWVudC5jcHAK PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PQotLS0gV2ViQ29yZS9wbGF0Zm9ybS9xdC9GcmFtZVF0Q2xpZW50LmNwcAkoUmV2 aXNpb24gMTc0NDApCisrKyBXZWJDb3JlL3BsYXRmb3JtL3F0L0ZyYW1lUXRDbGllbnQuY3BwCShB cmJlaXRza29waWUpCkBAIC02Nyw4ICs2NywxMyBAQCB2b2lkIEZyYW1lUXRDbGllbnREZWZhdWx0 OjpvcGVuVVJMKGNvbnN0CiAgICAgbV9mcmFtZS0+ZGlkT3BlblVSTCh1cmwpOwogICAgIG1fYXNz aWduZWRNaW1ldHlwZSA9IGZhbHNlOwogCisgICAgaWYgKCFtX2ZyYW1lLT5kb2N1bWVudCgpKQor ICAgICAgICBtX2ZyYW1lLT5jcmVhdGVFbXB0eURvY3VtZW50KCk7CisKKyAgICBBU1NFUlQobV9m cmFtZS0+ZG9jdW1lbnQoKSk7CisKICAgICBSZXNvdXJjZVJlcXVlc3QgcmVxdWVzdCh1cmwpOwot ICAgIFJlZlB0cjxSZXNvdXJjZUhhbmRsZT4gbG9hZGVyID0gUmVzb3VyY2VIYW5kbGU6OmNyZWF0 ZShyZXF1ZXN0LCB0aGlzLCBtX2ZyYW1lLT5kb2N1bWVudCgpID8gbV9mcmFtZS0+ZG9jdW1lbnQo KS0+ZG9jTG9hZGVyKCkgOiAwKTsKKyAgICBSZWZQdHI8UmVzb3VyY2VIYW5kbGU+IGxvYWRlciA9 IFJlc291cmNlSGFuZGxlOjpjcmVhdGUocmVxdWVzdCwgdGhpcywgbV9mcmFtZS0+ZG9jdW1lbnQo KS0+ZG9jTG9hZGVyKCkpOwogfQogCiB2b2lkIEZyYW1lUXRDbGllbnREZWZhdWx0OjpzdWJtaXRG b3JtKGNvbnN0IFN0cmluZyYgbWV0aG9kLCBjb25zdCBLVVJMJiB1cmwsIGNvbnN0IEZvcm1EYXRh KiBwb3N0RGF0YSkK