24665 2009-03-17 22:24:48 -0700 Hit testing assertion failure at Google Maps API example 2009-03-22 12:50:50 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Mac OS X 10.5 RESOLVED FIXED http://code.google.com/apis/maps/documentation/examples/map-infowindow.html NeedsReduction P2 Normal --- 1 zwarich simon.fraser mitz simon.fraser oldest_to_newest 114164 0 zwarich 2009-03-17 22:24:48 -0700 Hovering over the speech bubble at <http://code.google.com/apis/maps/documentation/examples/map-infowindow.html> causes an assertion failure in hit testing code. ASSERTION FAILED: !result.innerNode() (/Users/Cameron/WebKit/WebCore/rendering/RenderLayer.cpp:2407 bool WebCore::RenderLayer::hitTestContents(const WebCore::HitTestRequest&, WebCore::HitTestResult&, const WebCore::IntRect&, const WebCore::IntPoint&, WebCore::HitTestFilter) const) 114192 1 simon.fraser 2009-03-18 09:27:09 -0700 This means that someone returned true from nodeAtPoint but didn't set result.innerNode(). 114680 2 mitz 2009-03-21 20:12:18 -0700 *** Bug 24744 has been marked as a duplicate of this bug. *** 114715 3 28836 simon.fraser 2009-03-22 12:07:09 -0700 Created attachment 28836 Patch, changelog 114719 4 simon.fraser 2009-03-22 12:50:50 -0700 http://trac.webkit.org/changeset/41900 28836 2009-03-22 12:07:09 -0700 2009-03-22 12:16:18 -0700 Patch, changelog 24665_patch.txt text/plain 2186 simon.fraser ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg YTUwZTE2ZS4uYjFiNGViNSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi Q29yZS9DaGFuZ2VMb2cKQEAgLTEsNSArMSwxOSBAQAogMjAwOS0wMy0yMiAgU2ltb24gRnJhc2Vy ICA8c2ltb24uZnJhc2VyQGFwcGxlLmNvbT4KIAorICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg KE9PUFMhKS4KKyAgICAgICAgCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df YnVnLmNnaT9pZD0yNDY2NQorICAgICAgICAKKyAgICAgICAgSW1hZ2UtbWFwIGNvZGUgaW4gUmVu ZGVySW1hZ2UgY291bGQgcmVzdWx0IGluIFJlbmRlckltYWdlOjpub2RlQXRQb2ludCgpCisgICAg ICAgIHNldHRpbmcgSGl0VGVzdFJlc3VsdDo6aW5uZXJOb2RlKCksIGJ1dCByZXR1cm5pbmcgZmFs c2UsIHdoaWNoIHZpb2xhdGVzCisgICAgICAgIGhpdCB0ZXN0aW5nIHJ1bGVzLiBVc2UgYSB0ZW1w b3JhcnkgSGl0VGVzdFJlc3VsdCBzbyB0aGF0IHdlIG9ubHkgZmlsbCBpbgorICAgICAgICByZXN1 bHQgd2hlbiB3ZSBrbm93IHdlJ3ZlIGhpdC4KKworICAgICAgICAqIHJlbmRlcmluZy9SZW5kZXJJ bWFnZS5jcHA6CisgICAgICAgIChXZWJDb3JlOjpSZW5kZXJJbWFnZTo6bm9kZUF0UG9pbnQpOgor CisyMDA5LTAzLTIyICBTaW1vbiBGcmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgorCiAg ICAgICAgIFJldmlld2VkIGJ5IERhbiBCZXJuc3RlaW4KICAgICAgICAgCiAgICAgICAgIGh0dHBz Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yNDc0MwpkaWZmIC0tZ2l0IGEvV2Vi Q29yZS9yZW5kZXJpbmcvUmVuZGVySW1hZ2UuY3BwIGIvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVy SW1hZ2UuY3BwCmluZGV4IGRhMzA1ODEuLjgyZjM3NDEgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvcmVu ZGVyaW5nL1JlbmRlckltYWdlLmNwcAorKysgYi9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJJbWFn ZS5jcHAKQEAgLTQyMSw3ICs0MjEsOCBAQCBIVE1MTWFwRWxlbWVudCogUmVuZGVySW1hZ2U6Omlt YWdlTWFwKCkKIAogYm9vbCBSZW5kZXJJbWFnZTo6bm9kZUF0UG9pbnQoY29uc3QgSGl0VGVzdFJl cXVlc3QmIHJlcXVlc3QsIEhpdFRlc3RSZXN1bHQmIHJlc3VsdCwgaW50IF94LCBpbnQgX3ksIGlu dCBfdHgsIGludCBfdHksIEhpdFRlc3RBY3Rpb24gaGl0VGVzdEFjdGlvbikKIHsKLSAgICBib29s IGluc2lkZSA9IFJlbmRlclJlcGxhY2VkOjpub2RlQXRQb2ludChyZXF1ZXN0LCByZXN1bHQsIF94 LCBfeSwgX3R4LCBfdHksIGhpdFRlc3RBY3Rpb24pOworICAgIEhpdFRlc3RSZXN1bHQgdGVtcFJl c3VsdChyZXN1bHQucG9pbnQoKSk7CisgICAgYm9vbCBpbnNpZGUgPSBSZW5kZXJSZXBsYWNlZDo6 bm9kZUF0UG9pbnQocmVxdWVzdCwgdGVtcFJlc3VsdCwgX3gsIF95LCBfdHgsIF90eSwgaGl0VGVz dEFjdGlvbik7CiAKICAgICBpZiAoaW5zaWRlICYmIG5vZGUoKSkgewogICAgICAgICBpbnQgdHgg PSBfdHggKyB4KCk7CkBAIC00MzAsMTEgKzQzMSwxMyBAQCBib29sIFJlbmRlckltYWdlOjpub2Rl QXRQb2ludChjb25zdCBIaXRUZXN0UmVxdWVzdCYgcmVxdWVzdCwgSGl0VGVzdFJlc3VsdCYgcmVz dQogICAgICAgICBIVE1MTWFwRWxlbWVudCogbWFwID0gaW1hZ2VNYXAoKTsKICAgICAgICAgaWYg KG1hcCkgewogICAgICAgICAgICAgLy8gd2UncmUgYSBjbGllbnQgc2lkZSBpbWFnZSBtYXAKLSAg ICAgICAgICAgIGluc2lkZSA9IG1hcC0+bWFwTW91c2VFdmVudChfeCAtIHR4LCBfeSAtIHR5LCBJ bnRTaXplKGNvbnRlbnRXaWR0aCgpLCBjb250ZW50SGVpZ2h0KCkpLCByZXN1bHQpOwotICAgICAg ICAgICAgcmVzdWx0LnNldElubmVyTm9uU2hhcmVkTm9kZShub2RlKCkpOworICAgICAgICAgICAg aW5zaWRlID0gbWFwLT5tYXBNb3VzZUV2ZW50KF94IC0gdHgsIF95IC0gdHksIEludFNpemUoY29u dGVudFdpZHRoKCksIGNvbnRlbnRIZWlnaHQoKSksIHRlbXBSZXN1bHQpOworICAgICAgICAgICAg dGVtcFJlc3VsdC5zZXRJbm5lck5vblNoYXJlZE5vZGUobm9kZSgpKTsKICAgICAgICAgfQogICAg IH0KIAorICAgIGlmIChpbnNpZGUpCisgICAgICAgIHJlc3VsdCA9IHRlbXBSZXN1bHQ7CiAgICAg cmV0dXJuIGluc2lkZTsKIH0KIAo=