72788 2011-11-18 23:13:49 -0800 MHTML: the base url is not correct when the frame committed notification happens. 2011-11-19 00:45:23 -0800 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 jcivelli jcivelli abarth japhet webkit.review.bot oldest_to_newest 506337 0 jcivelli 2011-11-18 23:13:49 -0800 When loading a MHTML document, we set the base URL to the actual URL of the encoded document. This is done right after the notification that the frame was committed. It should be done before, so when the notification happens the base URL is correctly set on the document. 506340 1 115933 jcivelli 2011-11-18 23:21:41 -0800 Created attachment 115933 Patch 506357 2 115933 webkit.review.bot 2011-11-19 00:45:18 -0800 Comment on attachment 115933 Patch Clearing flags on attachment: 115933 Committed r100852: <http://trac.webkit.org/changeset/100852> 506358 3 webkit.review.bot 2011-11-19 00:45:23 -0800 All reviewed patches have been landed. Closing bug. 115933 2011-11-18 23:21:41 -0800 2011-11-19 00:45:18 -0800 Patch bug-72788-20111118232140.patch text/plain 1852 jcivelli U3VidmVyc2lvbiBSZXZpc2lvbjogMTAwODQ4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMDM5NzYzOTkxNTU2ZTZl ODhlNWViMDYyMTMxNjBmODdiZGM2MTJkNS4uMGZlNWNiY2Q1OGZiMDU0YjM5NmI2ZjdmZjFhMDRi Y2E5NTdkOThmMiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDExLTExLTE4ICBKYXkg Q2l2ZWxsaSAgPGpjaXZlbGxpQGNocm9taXVtLm9yZz4KKworICAgICAgICBXaGVuIGxvYWRpbmcg YSBNSFRNTCBkb2N1bWVudCwgbWFrZSBzdXJlIHdlIHNldCB0aGUgYmFzZSBVUkwgYmVmb3JlCisg ICAgICAgIHdlIG5vdGlmeSB0aGUgZnJhbWUgd2FzIGNvbW1pdHRlZCAoc28gdGhlIGRvY3VtZW50 IGhhcyB0aGUgcmlnaHQgYmFzZQorICAgICAgICBVUkwgd2hlbiB0aGUgbm90aWZpY2F0aW9uIGhh cHBlbnMpLiAKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk PTcyNzg4CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg KiBsb2FkZXIvRnJhbWVMb2FkZXIuY3BwOgorICAgICAgICAoV2ViQ29yZTo6RnJhbWVMb2FkZXI6 OnJlY2VpdmVkRmlyc3REYXRhKToKKwogMjAxMS0xMS0xOCAgVmluZWV0IENoYXVkaGFyeSAgPHZp bmVldC5jaGF1ZGhhcnlAbW90b3JvbGEuY29tPgogCiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr aXQub3JnL3Nob3dfYnVnLmNnaT9pZD03MjU5MQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUv bG9hZGVyL0ZyYW1lTG9hZGVyLmNwcCBiL1NvdXJjZS9XZWJDb3JlL2xvYWRlci9GcmFtZUxvYWRl ci5jcHAKaW5kZXggODRlZDRkMjRkZDhjODQwNDZiZGI0MTBiNzFkNmRhM2Y0NjNhM2VhNi4uYjI3 YjliOGYwZDU2NzE0NWFjOTY0MjQyMDQ0NmFlNTg1NmU2MTM1NSAxMDA2NDQKLS0tIGEvU291cmNl L1dlYkNvcmUvbG9hZGVyL0ZyYW1lTG9hZGVyLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9sb2Fk ZXIvRnJhbWVMb2FkZXIuY3BwCkBAIC01NzQsOSArNTc0LDYgQEAgdm9pZCBGcmFtZUxvYWRlcjo6 cmVjZWl2ZWRGaXJzdERhdGEoKQogICAgIGFjdGl2ZURvY3VtZW50TG9hZGVyKCktPndyaXRlcigp LT5iZWdpbih3b3JraW5nVVJMLCBmYWxzZSk7CiAgICAgYWN0aXZlRG9jdW1lbnRMb2FkZXIoKS0+ d3JpdGVyKCktPnNldERvY3VtZW50V2FzTG9hZGVkQXNQYXJ0T2ZOYXZpZ2F0aW9uKCk7CiAKLSAg ICBkaXNwYXRjaERpZENvbW1pdExvYWQoKTsKLSAgICBkaXNwYXRjaERpZENsZWFyV2luZG93T2Jq ZWN0c0luQWxsV29ybGRzKCk7Ci0KICNpZiBFTkFCTEUoTUhUTUwpCiAgICAgaWYgKG1fYXJjaGl2 ZSkgewogICAgICAgICAvLyBUaGUgb3JpZ2luIGlzIHRoZSBNSFRNTCBmaWxlLCB3ZSBuZWVkIHRv IHNldCB0aGUgYmFzZSBVUkwgdG8gdGhlIGRvY3VtZW50IGVuY29kZWQgaW4gdGhlIE1IVE1MIHNv CkBAIC01ODUsNiArNTgyLDkgQEAgdm9pZCBGcmFtZUxvYWRlcjo6cmVjZWl2ZWRGaXJzdERhdGEo KQogICAgIH0KICNlbmRpZgogCisgICAgZGlzcGF0Y2hEaWRDb21taXRMb2FkKCk7CisgICAgZGlz cGF0Y2hEaWRDbGVhcldpbmRvd09iamVjdHNJbkFsbFdvcmxkcygpOworCiAgICAgaWYgKG1fZG9j dW1lbnRMb2FkZXIpIHsKICAgICAgICAgU3RyaW5nV2l0aERpcmVjdGlvbiBwdGl0bGUgPSBtX2Rv Y3VtZW50TG9hZGVyLT50aXRsZSgpOwogICAgICAgICAvLyBJZiB3ZSBoYXZlIGEgdGl0bGUgbGV0 IHRoZSBXZWJWaWV3IGtub3cgYWJvdXQgaXQuCg==