62562 2011-06-13 10:47:57 -0700 Frame::visiblePositionForPoint returns null position if the point is over an imagemap 2011-06-13 15:38:49 -0700 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 1 enrica enrica oldest_to_newest 419661 0 enrica 2011-06-13 10:47:57 -0700 The innerNode() of an imageMap is an area, and doesn't have a renderer and therefore can't report a position. <rdar://problem/9313885> 419975 1 97019 enrica 2011-06-13 15:32:10 -0700 Created attachment 97019 Patch Postin this patch on behalf of Morgan Winer. 419980 2 97019 enrica 2011-06-13 15:33:40 -0700 Comment on attachment 97019 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=97019&action=review > Source/WebCore/ChangeLog:18 > + (WebCore::Frame::visiblePositionForPoint): Changin innerNode() to innerNonSharedNode(). Typo "Changin" 419989 3 enrica 2011-06-13 15:38:49 -0700 On behalf of Morgan Winer: Committed revision 88718. 97019 2011-06-13 15:32:10 -0700 2011-06-13 15:33:40 -0700 Patch morgan.txt text/plain 1775 enrica SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDg4NzA4KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMjIgQEAKKzIwMTEtMDYtMTMgICBNb3JnYW4g V2luZXIgIDxtd2luZXJAYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo T09QUyEpLgorCisgICAgICAgIEZyYW1lOjp2aXNpYmxlUG9zaXRpb25Gb3JQb2ludCByZXR1cm5z IG51bGwgcG9zaXRpb24gaWYgdGhlIHBvaW50IGlzIG92ZXIgYW4gaW1hZ2VtYXAuCisgICAgICAg IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD02MjU2MgorICAgICAgICA8 cmRhcjovL3Byb2JsZW0vOTMxMzg4NT4KKworICAgICAgICBUaGUgaW5uZXJOb2RlKCkgb2YgYW4g aW1hZ2VNYXAgaXMgYW4gYXJlYSwgYW5kIGRvZXNuJ3QgaGF2ZSBhIHJlbmRlcmVyIGFuZAorICAg ICAgICB0aGVyZWZvcmUgY2FuJ3QgcmVwb3J0IGEgcG9zaXRpb24uCisgICAgICAgIEV2ZXJ5IHJl c3VsdCdzIGlubmVyTm9uU2hhcmVkTm9kZSgpIHNob3VsZCBiZSB0aGUgc2FtZSBhcyB0aGUgaW5u ZXJOb2RlKCksCisgICAgICAgIGV4Y2VwdCBmb3IgaW1hZ2VNYXBzLCB3aGljaCB3aWxsIGhhdmUg dGhlIGFjdHVhbCBpbWFnZSBhcyBpdHMgaW5uZXJOb25TaGFyZWROb2RlKCksCisgICAgICAgIGFu ZCB3aWxsIHRoZXJlZm9yZSBoYXZlIGEgcmVuZGVyZXIgYW5kIGNhbiByZXBvcnQgYSBwb3NpdGlv bi4KKworICAgICAgICBUaGVyZSBhcmUgbm8gbGF5b3V0IHRlc3RzIGZvciB0aGlzLCBzaW5jZSBp dCBjYW4gYmUgdGVzdGVkIG9ubHkgYXQgQVBJIGxldmVsLgorCisgICAgICAgICogcGFnZS9GcmFt ZS5jcHA6CisgICAgICAgIChXZWJDb3JlOjpGcmFtZTo6dmlzaWJsZVBvc2l0aW9uRm9yUG9pbnQp OiBDaGFuZ2luIGlubmVyTm9kZSgpIHRvIGlubmVyTm9uU2hhcmVkTm9kZSgpLgorCiAyMDExLTA2 LTEzICBBbmRyZXkgS29zeWFrb3YgIDxjYXNlcUBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2 aWV3ZWQgYnkgRXJpYyBTZWlkZWwuCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wYWdlL0ZyYW1lLmNw cAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9wYWdlL0ZyYW1lLmNwcAkocmV2aXNpb24g ODg3MDIpCisrKyBTb3VyY2UvV2ViQ29yZS9wYWdlL0ZyYW1lLmNwcAkod29ya2luZyBjb3B5KQpA QCAtODU4LDcgKzg1OCw3IEBAIFN0cmluZyBGcmFtZTo6ZGlzcGxheVN0cmluZ01vZGlmaWVkQnlF bmMKIFZpc2libGVQb3NpdGlvbiBGcmFtZTo6dmlzaWJsZVBvc2l0aW9uRm9yUG9pbnQoY29uc3Qg SW50UG9pbnQmIGZyYW1lUG9pbnQpCiB7CiAgICAgSGl0VGVzdFJlc3VsdCByZXN1bHQgPSBldmVu dEhhbmRsZXIoKS0+aGl0VGVzdFJlc3VsdEF0UG9pbnQoZnJhbWVQb2ludCwgdHJ1ZSk7Ci0gICAg Tm9kZSogbm9kZSA9IHJlc3VsdC5pbm5lck5vZGUoKTsKKyAgICBOb2RlKiBub2RlID0gcmVzdWx0 LmlubmVyTm9uU2hhcmVkTm9kZSgpOwogICAgIGlmICghbm9kZSkKICAgICAgICAgcmV0dXJuIFZp c2libGVQb3NpdGlvbigpOwogICAgIFJlbmRlck9iamVjdCogcmVuZGVyZXIgPSBub2RlLT5yZW5k ZXJlcigpOwo=