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 72edc52631de1301441cd52394dd2d7c6d95d282..1efa6ea42e281b6232f7acdc35b4368031d7dd1d 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.
@@ -35,6 +36,9 @@ class NativeMenuWin : public MenuWrapper {
   virtual void RemoveMenuListener(MenuListener* listener);
   virtual void SetMinimumWidth(int width);
 
+  // MenuModel:Delegate called when a favicon is loaded from history.
+  virtual void OnIconChanged(int model_index) OVERRIDE;
+
  private:
   // IMPORTANT: Note about indices.
   //            Functions in this class deal in two index spaces: