147372 2015-07-28 13:44:53 -0700 bounds in InteractionInformationAtPosition should be always in main frame coordinate space. 2015-07-28 14:45:16 -0700 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 yongjun_zhang webkit-unassigned bdakin commit-queue enrica mitz sam thorton oldest_to_newest 1112846 0 yongjun_zhang 2015-07-28 13:44:53 -0700 When we prepare the bounds for InteractionInformationAtPosition in WebPageIOS.mm, it looks like we get the bounds in the current frame space. It should be in main frame space since WKContent in UIProcess expects it to be in the web view space. 1112848 1 yongjun_zhang 2015-07-28 13:45:15 -0700 <rdar://problem/21645007> 1112850 2 257677 yongjun_zhang 2015-07-28 13:49:06 -0700 Created attachment 257677 Patch. 1112884 3 257677 commit-queue 2015-07-28 14:45:12 -0700 Comment on attachment 257677 Patch. Clearing flags on attachment: 257677 Committed r187517: <http://trac.webkit.org/changeset/187517> 1112885 4 commit-queue 2015-07-28 14:45:16 -0700 All reviewed patches have been landed. Closing bug. 257677 2015-07-28 13:49:06 -0700 2015-07-28 14:45:12 -0700 Patch. bounds_sub_frame.patch text/plain 2058 yongjun_zhang Y29tbWl0IDBhZGMzMGRlZjZjZDY2ZGVkOGE3ZTJjOGQ2OTg0YzA5MjEwMmRhODcKQXV0aG9yOiBZ b25nanVuIFpoYW5nIDx5b25nanVuX3poYW5nQGFwcGxlLmNvbT4KRGF0ZTogICBUdWUgSnVsIDI4 IDEzOjQ0OjE3IDIwMTUgLTA3MDAKCiAgICBCb3VuZHMgaW4gSW50ZXJhY3Rpb25JbmZvcm1hdGlv bkF0UG9zaXRpb24gc2hvdWxkIGJlIGFsd2F5cyBpbiBtYWluIGZyYW1lIGNvb3JkaW5hdGUgc3Bh Y2UuCiAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTQ3MzcyCgpk aWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdDIvQ2hh bmdlTG9nCmluZGV4IDY4ZjY0YjguLmJmYjE4OGIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQy L0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNiBA QAorMjAxNS0wNy0yOCAgWW9uZ2p1biBaaGFuZyAgPHlvbmdqdW5femhhbmdAYXBwbGUuY29tPgor CisgICAgICAgIEJvdW5kcyBpbiBJbnRlcmFjdGlvbkluZm9ybWF0aW9uQXRQb3NpdGlvbiBzaG91 bGQgYmUgYWx3YXlzIGluIG1haW4gZnJhbWUgY29vcmRpbmF0ZSBzcGFjZS4KKyAgICAgICAgaHR0 cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE0NzM3MgorCisgICAgICAgIFdo ZW4gd2UgcHJlcGFyZSB0aGUgYm91bmRzIGZvciBJbnRlcmFjdGlvbkluZm9ybWF0aW9uQXRQb3Np dGlvbiwgd2Ugc2hvdWxkIGNvbnZlcnQgdGhlIHJlY3QgdG8KKyAgICAgICAgbWFpbiBmcmFtZSBz cGFjZSBzaW5jZSBXS0NvbnRlbnQgaW4gVUlQcm9jZXNzIGV4cGVjdHMgaXQgdG8gYmUgaW4gdGhl IHdlYiB2aWV3IHNwYWNlLgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor CisgICAgICAgICogV2ViUHJvY2Vzcy9XZWJQYWdlL2lvcy9XZWJQYWdlSU9TLm1tOgorICAgICAg ICAoV2ViS2l0OjpXZWJQYWdlOjpnZXRQb3NpdGlvbkluZm9ybWF0aW9uKTogQ29udmVydCB0aGUg Ym91bmRpbmcgcmVjdCB0byBtYWluIGZyYW1lIHNwYWNlIGlmIHRoZSBlbGVtZW50IGlzIGluc2lk ZSBhIHN1Yi1mcmFtZS4KKwogMjAxNS0wNy0yOCAgQ2FybG9zIEdhcmNpYSBDYW1wb3MgIDxjZ2Fy Y2lhQGlnYWxpYS5jb20+CiAKICAgICAgICAgW0dUS10gVXNlIGZhc3RNYWxsb2MgaW5zdGVhZCBv ZiBnX3NsaWNlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9XZWJQcm9jZXNzL1dlYlBhZ2Uv aW9zL1dlYlBhZ2VJT1MubW0gYi9Tb3VyY2UvV2ViS2l0Mi9XZWJQcm9jZXNzL1dlYlBhZ2UvaW9z L1dlYlBhZ2VJT1MubW0KaW5kZXggMDg2NDc2OC4uMjJmYTYzNyAxMDA2NDQKLS0tIGEvU291cmNl L1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL2lvcy9XZWJQYWdlSU9TLm1tCisrKyBiL1NvdXJj ZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9pb3MvV2ViUGFnZUlPUy5tbQpAQCAtMjI1Miw2 ICsyMjUyLDExIEBAIHZvaWQgV2ViUGFnZTo6Z2V0UG9zaXRpb25JbmZvcm1hdGlvbihjb25zdCBJ bnRQb2ludCYgcG9pbnQsIEludGVyYWN0aW9uSW5mb3JtYXRpCiAgICAgICAgICAgICAgICAgICAg IGluZm8uYm91bmRzID0gZG93bmNhc3Q8UmVuZGVySW1hZ2U+KCpyZW5kZXJlcikuYWJzb2x1dGVD b250ZW50UXVhZCgpLmVuY2xvc2luZ0JvdW5kaW5nQm94KCk7CiAgICAgICAgICAgICAgICAgZWxz ZQogICAgICAgICAgICAgICAgICAgICBpbmZvLmJvdW5kcyA9IHJlbmRlcmVyLT5hYnNvbHV0ZUJv dW5kaW5nQm94UmVjdCgpOworCisgICAgICAgICAgICAgICAgaWYgKCFyZW5kZXJlci0+ZG9jdW1l bnQoKS5mcmFtZSgpLT5pc01haW5GcmFtZSgpKSB7CisgICAgICAgICAgICAgICAgICAgIEZyYW1l VmlldyAqdmlldyA9IHJlbmRlcmVyLT5kb2N1bWVudCgpLmZyYW1lKCktPnZpZXcoKTsKKyAgICAg ICAgICAgICAgICAgICAgaW5mby5ib3VuZHMgPSB2aWV3LT5jb250ZW50c1RvUm9vdFZpZXcoaW5m by5ib3VuZHMpOworICAgICAgICAgICAgICAgIH0KICAgICAgICAgICAgIH0KICAgICAgICAgfQog ICAgIH0K