Separate Toolkit specific impl

chrome/browser/renderer_context_menu/render_view_context_menu.cc

Index: chrome/browser/renderer_context_menu/render_view_context_menu.cc
diff --git a/chrome/browser/renderer_context_menu/render_view_context_menu.cc b/chrome/browser/renderer_context_menu/render_view_context_menu.cc
index 8b1f54547a36a5301418e68b6e521c204cc1aece..4162feacb5a6fa10cafaea8fef02d19c3fae5609 100644
--- a/chrome/browser/renderer_context_menu/render_view_context_menu.cc
+++ b/chrome/browser/renderer_context_menu/render_view_context_menu.cc
@@ -459,11 +459,13 @@ RenderViewContextMenu::~RenderViewContextMenu() {
 
 void RenderViewContextMenu::Init() {
   InitMenu();
-  PlatformInit();
+  if (toolkit_delegate_)
+    toolkit_delegate_->Init(&menu_model_);
 }
 
 void RenderViewContextMenu::Cancel() {
-  PlatformCancel();
+  if (toolkit_delegate_)
+    toolkit_delegate_->Cancel();
 }
 
 static bool ExtensionPatternMatch(const extensions::URLPatternSet& patterns,
@@ -769,8 +771,12 @@ void RenderViewContextMenu::UpdateMenuItem(int command_id,
                                            bool enabled,
                                            bool hidden,
                                            const base::string16& label) {
-  // This function needs platform-specific implementation.
-  NOTIMPLEMENTED();
+  if (toolkit_delegate_) {
+    toolkit_delegate_->UpdateMenuItem(command_id,

[sky, 2014/07/31 21:23:43]

It seems like you're duplicating some of what MenuModelDelegate was intended
for. How about adding an ItemChanged to MenuModelDelegate and setters to
SimpleMenuModel for what this needs to change? I believe doing that would mean
no ToolkitDelegate.

[oshima, 2014/07/31 22:02:15]

I think it's quite different (partly due to the way RVConextMenu works).

RenderViewContextMenu has its own way to manage and update menu item, in async
way.
Menu's view status are managed in different ways (observers are controlling
views state).

It may be possible to rearchitecut the renderer context menu to use
MenuModelDelegate,
but that'd be huge task and the outside of the scope of this CL. If I'm
misunderstanding
something and there is a simpler way, let me know.

[sky, 2014/07/31 22:19:40]

I don't think it would be that much. If you make MenuModelAdapter a
MenuModelDegate and have RVCM::UpdateMenuItem call through to the delegate than
it'll work. For the cocoa side you could make RenderViewContextMenuCocoa
implement the MenuModelDelegate method. There may be a more central place there.
Not sure.

+                                      enabled,
+                                      hidden,
+                                      label);
+  }
 }
 
 RenderViewHost* RenderViewContextMenu::GetRenderViewHost() const {