119587 2013-08-08 11:56:44 -0700 WKBundleFrameCopyProvisionalURL crashes when there is no provisional loader 2013-08-08 12:23:21 -0700 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 ap ap oldest_to_newest 915539 0 ap 2013-08-08 11:56:44 -0700 WKBundleFrameCopyProvisionalURL crashes when there is no provisional loader. 915540 1 208359 ap 2013-08-08 11:58:24 -0700 Created attachment 208359 proposed fix 915551 2 208359 darin 2013-08-08 12:16:02 -0700 Comment on attachment 208359 proposed fix I’m going to assume that it’s better to return null rather than returning the committed string for that frame. 915556 3 ap 2013-08-08 12:23:21 -0700 Committed <http://trac.webkit.org/changeset/153839>. Yes, I think that it would be misleading to return committed URL when asked for provisional one. This also matches how -[WebFrame provisionalDataSource] works. 208359 2013-08-08 11:58:24 -0700 2013-08-08 12:16:01 -0700 proposed fix ProvisionalURL.txt text/plain 1425 ap SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDE1MzgzNykKKysrIFNvdXJjZS9XZWJLaXQyL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEzIEBACisyMDEzLTA4LTA4ICBBbGV4ZXkg UHJvc2t1cnlha292ICA8YXBAYXBwbGUuY29tPgorCisgICAgICAgIFdLQnVuZGxlRnJhbWVDb3B5 UHJvdmlzaW9uYWxVUkwgY3Jhc2hlcyB3aGVuIHRoZXJlIGlzIG5vIHByb3Zpc2lvbmFsIGxvYWRl cgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTE5NTg3 CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBXZWJQ cm9jZXNzL1dlYlBhZ2UvV2ViRnJhbWUuY3BwOiAoV2ViS2l0OjpXZWJGcmFtZTo6cHJvdmlzaW9u YWxVUkwpOiBBZGRlZCBhIG51bGwKKyAgICAgICAgY2hlY2ssIGxpa2Ugd2UgYWxyZWFkeSBoYXZl IGluIFdlYkZyYW1lOjp1cmwoKS4KKwogMjAxMy0wOC0wNyAgQW50dGkgS29pdmlzdG8gIDxhbnR0 aUBhcHBsZS5jb20+CiAKICAgICAgICAgVHJ5IHRvIGZpeCBFRkwgYnVpbGQuCkluZGV4OiBTb3Vy Y2UvV2ViS2l0Mi9XZWJQcm9jZXNzL1dlYlBhZ2UvV2ViRnJhbWUuY3BwCj09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t IFNvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9XZWJGcmFtZS5jcHAJKHJldmlzaW9u IDE1MzgyOCkKKysrIFNvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9XZWJGcmFtZS5j cHAJKHdvcmtpbmcgY29weSkKQEAgLTY4MSw3ICs2ODEsMTEgQEAgU3RyaW5nIFdlYkZyYW1lOjpw cm92aXNpb25hbFVSTCgpIGNvbnN0CiAgICAgaWYgKCFtX2NvcmVGcmFtZSkKICAgICAgICAgcmV0 dXJuIFN0cmluZygpOwogCi0gICAgcmV0dXJuIG1fY29yZUZyYW1lLT5sb2FkZXIoKS0+cHJvdmlz aW9uYWxEb2N1bWVudExvYWRlcigpLT51cmwoKS5zdHJpbmcoKTsKKyAgICBEb2N1bWVudExvYWRl ciogcHJvdmlzaW9uYWxEb2N1bWVudExvYWRlciA9IG1fY29yZUZyYW1lLT5sb2FkZXIoKS0+cHJv dmlzaW9uYWxEb2N1bWVudExvYWRlcigpOworICAgIGlmICghcHJvdmlzaW9uYWxEb2N1bWVudExv YWRlcikKKyAgICAgICAgcmV0dXJuIFN0cmluZygpOworCisgICAgcmV0dXJuIHByb3Zpc2lvbmFs RG9jdW1lbnRMb2FkZXItPnVybCgpLnN0cmluZygpOwogfQogCiBTdHJpbmcgV2ViRnJhbWU6OnN1 Z2dlc3RlZEZpbGVuYW1lRm9yUmVzb3VyY2VXaXRoVVJMKGNvbnN0IEtVUkwmIHVybCkgY29uc3QK