RESOLVED FIXED 114860
[BlackBerry] Remove implementation of ContextMenu classes 
https://bugs.webkit.org/show_bug.cgi?id=114860
Summary [BlackBerry] Remove implementation of ContextMenu classes
Alberto Garcia
Reported 2013-04-19 02:16:54 PDT
Although in the BlackBerry port a PlatformMenuDescription is actually a void*, the definition of the method should match that of the parent class.
Attachments
Patch (2.43 KB, patch)
2013-04-19 02:17 PDT, Alberto Garcia 		no flags
DetailsFormatted DiffDiff
Patch (4.97 KB, patch)
2013-04-19 05:39 PDT, Alberto Garcia 		no flags
DetailsFormatted DiffDiff
Patch (18.27 KB, patch)
2013-06-10 02:21 PDT, Alberto Garcia 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Alberto Garcia
Comment 1 2013-04-19 02:17:53 PDT
Created attachment 198827 [details] Patch
Alberto Garcia
Comment 2 2013-04-19 02:35:16 PDT
Comment on attachment 198827 [details] Patch This patch is actually incomplete, I'll upload a new one.
Alberto Garcia
Comment 3 2013-04-19 05:35:09 PDT
Updating bug title since a few more changes are needed.
Alberto Garcia
Comment 4 2013-04-19 05:39:22 PDT
Created attachment 198845 [details] Patch
Alberto Garcia
Comment 5 2013-06-10 02:19:17 PDT
We are finally not going to use this, so we can safely remove this code.
Alberto Garcia
Comment 6 2013-06-10 02:21:11 PDT
Created attachment 204143 [details] Patch
Rob Buis
Comment 7 2013-06-14 12:37:40 PDT
Comment on attachment 204143 [details] Patch LGTM. Will obviously bring a small speedup in build time :)
WebKit Commit Bot
Comment 8 2013-06-14 13:03:01 PDT
Comment on attachment 204143 [details] Patch Clearing flags on attachment: 204143 Committed r151603: <http://trac.webkit.org/changeset/151603>
WebKit Commit Bot
Comment 9 2013-06-14 13:03:05 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.