ServicesOverlayController menus show up in the wrong place.
This is despite the fact that they use the context menu mechanism which does display context menus in the right place.
The difference is that context menu clicks undergo a hit test and have their coordinate translated to window-space.
We need to do the same for Services.
Created attachment 234496 [details]
Comment on attachment 234496 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=234496&action=review
> +void ServicesOverlayController::handleClick(const WebCore::IntPoint& clickPoint)
This seems wrong. The incoming point is always in a consistent coordinate space (though I'm not sure at the moment which one), not dependent on what frame you click on. Am I wrong and this somehow actually works in subframes?
Discussed with Tim in person - This patch wasn't *wrong*, it was just more work than was needed.
Simpler patch coming up, that also includes a fix for a subframe crasher that showed up in testing subframes.
Created attachment 234499 [details]
Confirmed with various main frame scroll positions, subframe scroll positions, and page zoom situations, that this patch gives the same coordinates as the hit test patch.
Comment on attachment 234499 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=234499&action=review
> + Page* page = m_webPage->corePage();
Please use WebPage::mainFrameView() instead of all of this.
Re-opened since this is blocked by bug 134783
(In reply to comment #7)
> Re-opened since this is blocked by bug 134783
That was just a test of webkitbot. All is well.