Native menu implementation for bug 5679. Followup to http://codereview.chromium.org/2928005/

views/controls/menu/native_menu_win.h

Index: views/controls/menu/native_menu_win.h
diff --git a/views/controls/menu/native_menu_win.h b/views/controls/menu/native_menu_win.h
index eb28bbed9fe1ed6a714ae91d9a4f139557b70fed..1ab169ea3936f45c0c8b779cece38c9ec304f622 100644
--- a/views/controls/menu/native_menu_win.h
+++ b/views/controls/menu/native_menu_win.h
@@ -16,7 +16,8 @@ namespace views {
 
 // A Windows implementation of MenuWrapper.
 // TODO(beng): rename to MenuWin once the old class is dead.
-class NativeMenuWin : public MenuWrapper {
+class NativeMenuWin : public MenuWrapper,
+                      public ui::MenuModelDelegate {
  public:
   // Construct a NativeMenuWin, with a model and delegate. If |system_menu_for|
   // is non-NULL, the NativeMenuWin wraps the system menu for that window.
@@ -34,6 +35,8 @@ class NativeMenuWin : public MenuWrapper {
   virtual void AddMenuListener(MenuListener* listener);
   virtual void RemoveMenuListener(MenuListener* listener);
   virtual void SetMinimumWidth(int width);
+  // MenuModel:Delegate called when a favicon is loaded from history.

[sky, 2011/03/24 22:39:29]

Newline between 37/38. Also, add OVERRIDE.

+  virtual void OnIconChanged(int model_index);
 
  private:
   // IMPORTANT: Note about indices.