Bug 113958 - Add SPI to get a copy of the context menu at a given point.
Summary: Add SPI to get a copy of the context menu at a given point.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Brady Eidson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2013-04-04 13:59 PDT by Brady Eidson
Modified: 2013-04-05 09:25 PDT (History)
3 users (show)

See Also:

Attachments
Patch v1 - First swipe (4.03 KB, patch)
2013-04-04 14:01 PDT, Brady Eidson 		aestes: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Brady Eidson 2013-04-04 13:59:03 PDT 
Add SPI to get a copy of the context menu at a given point.

This is similar to [NSView menuForEvent:] in Cocoa/WebKit1

In radar as <rdar://problem/13450908>
Comment 1 Brady Eidson 2013-04-04 14:01:44 PDT 
Created attachment 196525 [details]
Patch v1 - First swipe
Comment 2 Andy Estes 2013-04-04 16:23:53 PDT 
Comment on attachment 196525 [details]
Patch v1 - First swipe

View in context: https://bugs.webkit.org/attachment.cgi?id=196525&action=review

r=me with a comment

> Source/WebKit2/WebProcess/InjectedBundle/API/c/WKBundlePagePrivate.h:81
> +WK_EXPORT WKArrayRef WKBundlePageCopyContextMenuAtPoint(WKBundlePageRef, WKPoint);

We should be clear what coordinate space we expect the point to be in, since your code in WebPage::contextMenuAtPoint() creates a mouse event assuming screen coordinate space. We decided on WKBundlePageCopyContextMenuAtPointOnScreen() over IRC.
Comment 3 Brady Eidson 2013-04-05 09:25:12 PDT 
http://trac.webkit.org/changeset/147752