Summary: | [GTK][WK2] Update ContextMenu Download API | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brian Holt <brian.holt> | ||||
Component: | WebKitGTK | Assignee: | Brian Holt <brian.holt> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cgarcia, commit-queue, gustavo, mrobinson, pnormand | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Brian Holt
2013-09-05 02:10:00 PDT
Created attachment 210627 [details]
Patch
Comment on attachment 210627 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=210627&action=review > Source/WebKit2/UIProcess/API/gtk/WebKitContextMenuActions.h:75 > + * @WEBKIT_CONTEXT_MENU_ACTION_DOWNLOAD_VIDEO_TO_DISK: Download video to disk. Since 2.2 Unfortunately this doesn't result in the enum appearing under the Since 2.2 section of the docs... Perhaps there is a different way, or it doesn't matter? > Source/WebKit2/UIProcess/API/gtk/tests/TestContextMenu.cpp:324 > + iter = checkCurrentItemIsStockActionAndGetNext(iter, WEBKIT_CONTEXT_MENU_ACTION_ENTER_VIDEO_FULLSCREEN, Visible); This surprised me, but without it the test will not pass! Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API Comment on attachment 210627 [details] Patch Clearing flags on attachment: 210627 Committed r155345: <http://trac.webkit.org/changeset/155345> All reviewed patches have been landed. Closing bug. |