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

chrome/browser/ui/gtk/menu_gtk.h

Index: chrome/browser/ui/gtk/menu_gtk.h
diff --git a/chrome/browser/ui/gtk/menu_gtk.h b/chrome/browser/ui/gtk/menu_gtk.h
index 381993bbba5d367ffd4055807906588d3320b9c0..65d7ac6afe2936848d975be50059e6fce5186129 100644
--- a/chrome/browser/ui/gtk/menu_gtk.h
+++ b/chrome/browser/ui/gtk/menu_gtk.h
@@ -13,16 +13,16 @@
 
 #include "base/task.h"
 #include "ui/base/gtk/gtk_signal.h"
+#include "ui/base/models/menu_model.h"
 #include "ui/gfx/point.h"
 
 class SkBitmap;
 
 namespace ui {
 class ButtonMenuItemModel;
-class MenuModel;
 }
 
-class MenuGtk {
+class MenuGtk : public ui::MenuModelDelegate {
  public:
   // Delegate class that lets another class control the status of the menu.
   class Delegate {
@@ -98,6 +98,12 @@ class MenuGtk {
   // Closes the menu.
   void Cancel();
 
+  // From ui::MenuModelDelegate, called when a favicon is loaded from history.
+  virtual void OnIconChanged(int model_index);

[Avi (use Gerrit), 2011/03/25 00:19:35]

OVERRIDE

+
+  // Update the icon for a particular menu_item (widget).
+  static void UpdateMenuIcon(GtkWidget* widget, gpointer userdata);

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

Why is this public?

+
   // Repositions the menu to be right under the button.  Alignment is set as
   // object data on |void_widget| with the tag "left_align".  If "left_align"
   // is true, it aligns the left side of the menu with the left side of the