IsCommandIdEnabled should not fall though if the base did handle the command id

components/renderer_context_menu/render_view_context_menu_base.h

Index: components/renderer_context_menu/render_view_context_menu_base.h
diff --git a/components/renderer_context_menu/render_view_context_menu_base.h b/components/renderer_context_menu/render_view_context_menu_base.h
index 9d4ac636c0fc38bc8f855d37c2f6ae7ae8604ed3..3179caaeebe2dbf4005551946124ed7e4887be8d 100644
--- a/components/renderer_context_menu/render_view_context_menu_base.h
+++ b/components/renderer_context_menu/render_view_context_menu_base.h
@@ -78,9 +78,12 @@ class RenderViewContextMenuBase : public ui::SimpleMenuModel::Delegate,
   const ui::SimpleMenuModel& menu_model() const { return menu_model_; }
   const content::ContextMenuParams& params() const { return params_; }
 
+  // Sets the state of the command id in |enabled|. The function
+  // returns true if the command id is handled.
+  bool GetCommandIdEnabled(int command_id, bool* enabled) const;

[sky, 2014/08/08 16:36:23]

The description you have here and the name is mildly confusing. How about:

// Returns true if the specified command id is known and valid for this menu. If
// the command is known |enabled| is set to indicate if the command is enabled.
bool IsCommandIdKnown(int command_id, bool* enabled);

[oshima, 2014/08/08 17:14:17]

Done.

+
   // SimpleMenuModel::Delegate implementation.
   virtual bool IsCommandIdChecked(int command_id) const OVERRIDE;
-  virtual bool IsCommandIdEnabled(int command_id) const OVERRIDE;
   virtual void ExecuteCommand(int command_id, int event_flags) OVERRIDE;
   virtual void MenuWillShow(ui::SimpleMenuModel* source) OVERRIDE;
   virtual void MenuClosed(ui::SimpleMenuModel* source) OVERRIDE;