73709 2011-12-02 15:16:24 -0800 Build fix for SubresourceLoader when build with Core Foundation 2011-12-02 15:55:32 -0800 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 benjamin benjamin ddkilzer joepeck oldest_to_newest 514217 0 117703 benjamin 2011-12-02 15:16:24 -0800 Created attachment 117703 Patch Reviewed by NOBODY (OOPS!). The patch r100311 removed SubresourceClient and merged it in SubresourceLoader. Consequently, m_client does not exist anymore and there is no need to do the check before invoking didReceiveData(). * loader/cf/SubresourceLoaderCF.cpp: (WebCore::SubresourceLoader::didReceiveDataArray): --- 2 files changed, 13 insertions(+), 2 deletions(-) 514235 1 117703 ddkilzer 2011-12-02 15:41:52 -0800 Comment on attachment 117703 Patch r=me 514245 2 benjamin 2011-12-02 15:55:32 -0800 Committed r101874: <http://trac.webkit.org/changeset/101874> 117703 2011-12-02 15:16:24 -0800 2011-12-02 15:41:51 -0800 Patch 20111202151623.patch text/plain 1790 benjamin U3VidmVyc2lvbiBSZXZpc2lvbjogMTAxODYyCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZWEyZWY5YWRjMjE3YjIz ZDg4ZWZhODMzZDBmYTI2ODRmM2QzYTI5ZS4uZWVkYjc1ZjJhMmY0MjNkNmMyMzAzMzkyNmJiZjJi OTI4YjBlODBlYSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDExLTEyLTAyICBCZW5q YW1pbiBQb3VsYWluICA8YmVuamFtaW5Ad2Via2l0Lm9yZz4KKworICAgICAgICBCdWlsZCBmaXgg Zm9yIFN1YnJlc291cmNlTG9hZGVyIHdoZW4gYnVpbGQgd2l0aCBDb3JlIEZvdW5kYXRpb24KKwor ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBUaGUgcGF0Y2gg cjEwMDMxMSByZW1vdmVkIFN1YnJlc291cmNlQ2xpZW50IGFuZCBtZXJnZWQgaXQgaW4gU3VicmVz b3VyY2VMb2FkZXIuCisgICAgICAgIENvbnNlcXVlbnRseSwgbV9jbGllbnQgZG9lcyBub3QgZXhp c3QgYW55bW9yZSBhbmQgdGhlcmUgaXMgbm8gbmVlZCB0byBkbyB0aGUgY2hlY2sKKyAgICAgICAg YmVmb3JlIGludm9raW5nIGRpZFJlY2VpdmVEYXRhKCkuCisKKyAgICAgICAgKiBsb2FkZXIvY2Yv U3VicmVzb3VyY2VMb2FkZXJDRi5jcHA6CisgICAgICAgIChXZWJDb3JlOjpTdWJyZXNvdXJjZUxv YWRlcjo6ZGlkUmVjZWl2ZURhdGFBcnJheSk6CisKIDIwMTEtMTItMDIgIERhbiBCZXJuc3RlaW4g IDxtaXR6QGFwcGxlLmNvbT4KIAogICAgICAgICA8cmRhcjovL3Byb2JsZW0vMTA1MjA2NzA+IFJF R1JFU1NJT04gKHI5MTczOCk6IGRpZEZpbmlzaExvYWQgaXMgY2FsbGVkIGJlZm9yZSBjdXN0b20g Zm9udHMgaGF2ZSBmaW5pc2hlZCBsb2FkaW5nCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9s b2FkZXIvY2YvU3VicmVzb3VyY2VMb2FkZXJDRi5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9sb2FkZXIv Y2YvU3VicmVzb3VyY2VMb2FkZXJDRi5jcHAKaW5kZXggNTYyZDEyY2I1YWY0NDY3ZmZjMDUyYWJl YTlkNGFjYjg3ZmFkY2NhMi4uMTVhYjg4N2Y4NjBlMDYwYzRmNWY3MDUzN2UwNGI0MzNkYmIwNGEx NyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvbG9hZGVyL2NmL1N1YnJlc291cmNlTG9hZGVy Q0YuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL2xvYWRlci9jZi9TdWJyZXNvdXJjZUxvYWRlckNG LmNwcApAQCAtNDEsOCArNDEsNiBAQCB2b2lkIFN1YnJlc291cmNlTG9hZGVyOjpkaWRSZWNlaXZl RGF0YUFycmF5KENGQXJyYXlSZWYgZGF0YUFycmF5KQogICAgIGlmICghbV9sb2FkaW5nTXVsdGlw YXJ0Q29udGVudCkgewogICAgICAgICBDRkluZGV4IGFycmF5Q291bnQgPSBDRkFycmF5R2V0Q291 bnQoZGF0YUFycmF5KTsKICAgICAgICAgZm9yIChDRkluZGV4IGkgPSAwOyBpIDwgYXJyYXlDb3Vu dDsgKytpKSAgewotICAgICAgICAgICAgaWYgKCFtX2NsaWVudCkKLSAgICAgICAgICAgICAgICBi cmVhazsKICAgICAgICAgICAgIENGRGF0YVJlZiBkYXRhID0gcmVpbnRlcnByZXRfY2FzdDxDRkRh dGFSZWY+KENGQXJyYXlHZXRWYWx1ZUF0SW5kZXgoZGF0YUFycmF5LCBpKSk7CiAgICAgICAgICAg ICBkaWRSZWNlaXZlRGF0YShyZWludGVycHJldF9jYXN0PGNvbnN0IGNoYXIgKj4oQ0ZEYXRhR2V0 Qnl0ZVB0cihkYXRhKSksIHN0YXRpY19jYXN0PGludD4oQ0ZEYXRhR2V0TGVuZ3RoKGRhdGEpKSwg LTEsIGZhbHNlKTsKICAgICAgICAgfQo=