Add identifier strings for a bunch of context menu items
Created attachment 265489 [details] Patch
Comment on attachment 265489 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=265489&action=review > Source/WebKit2/UIProcess/API/Cocoa/WKMenuItemIdentifiers.mm:35 > +NSString * const _WKMenuItemIdentifierDownloadLinkedFile = @"WKMenuItemIdentifierDownloadLinkedFile"; Maybe we should use resource instead of file? > Source/WebKit2/UIProcess/API/Cocoa/WKMenuItemIdentifiers.mm:48 > +NSString * const _WKMenuItemIdentifierShowHideMediaControls = @"WKMenuItemIdentifierShowHideMediaControls"; > +NSString * const _WKMenuItemIdentifierToggleFullScreen = @"WKMenuItemIdentifierToggleFullScreen"; What’s the difference between ShowHide and Toggle?
Committed r192445: <http://trac.webkit.org/changeset/192445>
This broke the Mac build (and EWS is red!) Please fix.
Anders is not available to fix, rolling out.
Re-opened since this is blocked by bug 151291
Created attachment 265594 [details] Patch
Created attachment 265602 [details] Patch
Committed r192481: <http://trac.webkit.org/changeset/192481>
(In reply to comment #9) > Committed r192481: <http://trac.webkit.org/changeset/192481> It broke the Apple Mac cmake build.
Comment on attachment 265602 [details] Patch Cleared review? from attachment 265602 [details] so that this bug does not appear in http://webkit.org/pending-review. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).
(In reply to comment #10) > (In reply to comment #9) > > Committed r192481: <http://trac.webkit.org/changeset/192481> > > It broke the Apple Mac cmake build. Just to document, it was fixed by https://trac.webkit.org/changeset/192502