80543 2012-03-07 14:36:08 -0800 Assertion failure in ResourceHandle::setDefersLoading(): d->m_defersLoading != defers 2012-03-09 16:46:00 -0800 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 psolanki psolanki ap japhet koivisto psolanki simon.fraser webkit.review.bot oldest_to_newest 573248 0 psolanki 2012-03-07 14:36:08 -0800 It's possible to hit the assertion in ResourceHandle::setDefersLoading() in the following sequence - We create a ResourceLoader (subresource loader) with defersLoading = false - But the ResourceLoader hasn't been added to the subresource list in DocumentLoader and instead is sitting the pending requests list in ResourceLoadScheduler. - Page::setDefersLoading(true) gets called which changes the flag on all subresources except those sitting in the pending requests list. - Later, the request in the pending request list get started which adds them into the subresource list in DocumentLoader - A subsequent call to Page::setDefersLoading() triggers the assert. 573249 1 psolanki 2012-03-07 14:36:27 -0800 <rdar://problem/10963482> 573277 2 130704 psolanki 2012-03-07 15:03:07 -0800 Created attachment 130704 Patch 574391 3 130704 ggaren 2012-03-08 16:54:54 -0800 Comment on attachment 130704 Patch r=me 575344 4 psolanki 2012-03-09 16:45:35 -0800 Committed r110349: <http://trac.webkit.org/changeset/110349> 130704 2012-03-07 15:03:07 -0800 2012-03-09 16:46:00 -0800 Patch bug-80543-20120307150306.patch text/plain 1765 psolanki U3VidmVyc2lvbiBSZXZpc2lvbjogMTEwMDk4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYzg3N2EwYTExZTcyNDJh ZGYyODNmMjYzNjE0ZTM4Yzk1N2RhYWMwZi4uYWFlMGYwZjRjYjdkYTg1NzhiMDc1MTU3ZTY3MmYz ODYyMGMyMDQyZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDEyLTAzLTA3ICBQcmF0 aWsgU29sYW5raSAgPHBzb2xhbmtpQGFwcGxlLmNvbT4KKworICAgICAgICBBc3NlcnRpb24gZmFp bHVyZSBpbiBSZXNvdXJjZUhhbmRsZTo6c2V0RGVmZXJzTG9hZGluZygpOiBkLT5tX2RlZmVyc0xv YWRpbmcgIT0gZGVmZXJzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn LmNnaT9pZD04MDU0MworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisg ICAgICAgIFJlc2V0IG1fZGVmZXJzTG9hZGluZyBmbGFnIHRvIHRoZSB2YWx1ZSBmcm9tIFBhZ2U6 OmRlZmVyc0NhbGxiYWNrcygpIGluCisgICAgICAgIFJlc291cmNlTG9hZGVyOjppbml0KCkuIFRo aXMgaXMgYmVjYXVzZSB0aGUgcmVzb3VyY2UgY291bGQgaGF2ZSBiZWVuIGluIHRoZSBwZW5kaW5n IHJlcXVlc3RzCisgICAgICAgIHF1ZXVlIGluIFJlc291cmNlTG9hZFNjaGVkdWxlciBhbmQgd291 bGQgaGF2ZSBtaXNzZWQgb3V0IG9uIHN0YXRlIGNoYW5nZXMgdG8gdGhpcyBmbGFnCisgICAgICAg IGhhcHBlbmluZyBmcm9tIERvY3VtZW50TG9hZGVyOjpzZXREZWZlcnNDYWxsYmFja3MoKS4KKwor ICAgICAgICAqIGxvYWRlci9SZXNvdXJjZUxvYWRlci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpS ZXNvdXJjZUxvYWRlcjo6aW5pdCk6CisKIDIwMTItMDMtMDcgIEVyaWMgQ2FybHNvbiAgPGVyaWMu Y2FybHNvbkBhcHBsZS5jb20+CiAKICAgICAgICAgSHRtbDUgdmlkZW8gZWxlbWVudCBVc2VyYWdl bnQgc3RyaW5nIGlzIFF1aWNrdGltZQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvbG9hZGVy L1Jlc291cmNlTG9hZGVyLmNwcCBiL1NvdXJjZS9XZWJDb3JlL2xvYWRlci9SZXNvdXJjZUxvYWRl ci5jcHAKaW5kZXggNTgzNWYxZWFjY2FmMjlmZGY0Y2UxMGY4YTNkYmZhNDA3YWVjMzMxNi4uZDhm ZTdkYWIwOGM5OGQ4OGUxYjVkNWM4OTkzZjNlYTQyNDEwOGFiYSAxMDA2NDQKLS0tIGEvU291cmNl L1dlYkNvcmUvbG9hZGVyL1Jlc291cmNlTG9hZGVyLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9s b2FkZXIvUmVzb3VyY2VMb2FkZXIuY3BwCkBAIC0xMTMsNiArMTEzLDcgQEAgYm9vbCBSZXNvdXJj ZUxvYWRlcjo6aW5pdChjb25zdCBSZXNvdXJjZVJlcXVlc3QmIHIpCiAgICAgCiAgICAgUmVzb3Vy Y2VSZXF1ZXN0IGNsaWVudFJlcXVlc3Qocik7CiAgICAgCisgICAgbV9kZWZlcnNMb2FkaW5nID0g bV9mcmFtZS0+cGFnZSgpLT5kZWZlcnNMb2FkaW5nKCk7CiAgICAgaWYgKG1fb3B0aW9ucy5zZWN1 cml0eUNoZWNrID09IERvU2VjdXJpdHlDaGVjayAmJiAhbV9mcmFtZS0+ZG9jdW1lbnQoKS0+c2Vj dXJpdHlPcmlnaW4oKS0+Y2FuRGlzcGxheShjbGllbnRSZXF1ZXN0LnVybCgpKSkgewogICAgICAg ICBGcmFtZUxvYWRlcjo6cmVwb3J0TG9jYWxMb2FkRmFpbGVkKG1fZnJhbWUuZ2V0KCksIGNsaWVu dFJlcXVlc3QudXJsKCkuc3RyaW5nKCkpOwogICAgICAgICByZWxlYXNlUmVzb3VyY2VzKCk7Cg==