added CTRL+Click and SHIFT+Click handler for context menu, Back and Forward.

ui/base/models/simple_menu_model.h

Index: ui/base/models/simple_menu_model.h
diff --git a/ui/base/models/simple_menu_model.h b/ui/base/models/simple_menu_model.h
index e8ccb40afec92d21af4767801331d60381f45c60..96a4294d733379dcb00a45887c65ca11c6351ba4 100644
--- a/ui/base/models/simple_menu_model.h
+++ b/ui/base/models/simple_menu_model.h
@@ -48,6 +48,8 @@ class SimpleMenuModel : public MenuModel {
 
     // Performs the action associated with the specified command id.
     virtual void ExecuteCommand(int command_id) = 0;
+    virtual void ExecuteCommandWithDisposition(int command_id,
+                                               int disposition);
 
     // Notifies the delegate that the menu is about to show.
     virtual void MenuWillShow();
@@ -122,6 +124,7 @@ class SimpleMenuModel : public MenuModel {
   virtual bool IsVisibleAt(int index) const OVERRIDE;
   virtual void HighlightChangedTo(int index) OVERRIDE;
   virtual void ActivatedAt(int index) OVERRIDE;
+  virtual void ActivatedAtWithDisposition(int index, int disposition) OVERRIDE;

[Ben Goodger (Google), 2011/05/13 19:46:26]

This is an implicit coupling between this model code and a WebKit concept -
WindowOpenDisposition. Even if you don't expose the type here, the
calling/receiving code expects it.

Why does the model need to know about the disposition?

In fact, who even calls this method? I don't see any callers in this CL.

   virtual MenuModel* GetSubmenuModelAt(int index) const OVERRIDE;
   virtual void MenuWillShow() OVERRIDE;
   virtual void MenuClosed() OVERRIDE;