Implement the Chromium-specific part of WebCore context menus with submenus

Implement the Chromium-specific part of WebCore context menus with submenus. The WebKit counterpart is tracked at https://bugs.webkit.org/show_bug.cgi?id=86625


BUG=128351
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=138540

[apavlov, 2012-05-18 17:33:29]

Folks,

Could you have a look at this patch? Its WebKit counterpart is up for review at
https://bugs.webkit.org/show_bug.cgi?id=86625. The entire change is intended to
allow WebCore display ContextMenus with submenus (which is necessary for the
DevTools).

[pfeldman, 2012-05-22 14:22:29]

https://chromiumcodereview.appspot.com/10411002/diff/1/chrome/browser/tab_con...
File chrome/browser/tab_contents/render_view_context_menu.cc (right):

https://chromiumcodereview.appspot.com/10411002/diff/1/chrome/browser/tab_con...
chrome/browser/tab_contents/render_view_context_menu.cc:185: case
WebMenuItem::GROUP:
why did this change?

[pfeldman, 2012-05-22 14:49:44]

lgtm

[commit-bot: I haz the power, 2012-05-23 09:16:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/apavlov@chromium.org/10411002/4001

[commit-bot: I haz the power, 2012-05-23 09:16:15]

Presubmit check for 10411002-4001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    webkit

[apavlov, 2012-05-23 09:18:55]

Dear src/webkit OWNERS,

Can you please lgtm this change? Thanks!

[tony, 2012-05-23 16:12:59]

LGTM

[commit-bot: I haz the power, 2012-05-23 16:21:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/apavlov@chromium.org/10411002/4001

[commit-bot: I haz the power, 2012-05-23 18:39:59]

Change committed as 138540