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

chrome/browser/ui/cocoa/toolbar/back_forward_menu_controller.h

Index: chrome/browser/ui/cocoa/toolbar/back_forward_menu_controller.h
diff --git a/chrome/browser/ui/cocoa/toolbar/back_forward_menu_controller.h b/chrome/browser/ui/cocoa/toolbar/back_forward_menu_controller.h
index 4a978b5f8752801fc93a1f42d158b0865eda4397..ffb8573e41e16fc6a48bd61abe5b06e428c43d5d 100644
--- a/chrome/browser/ui/cocoa/toolbar/back_forward_menu_controller.h
+++ b/chrome/browser/ui/cocoa/toolbar/back_forward_menu_controller.h
@@ -14,6 +14,11 @@
 
 @class MenuButton;
 
+namespace BackForwardMenuControllerInternal {
+// A C++ delegate that handles favicons as they are loaded from history.
+class MenuDelegate;
+}
+
 typedef BackForwardMenuModel::ModelType BackForwardMenuType;
 const BackForwardMenuType BACK_FORWARD_MENU_TYPE_BACK =
     BackForwardMenuModel::BACKWARD_MENU;
@@ -29,6 +34,8 @@ const BackForwardMenuType BACK_FORWARD_MENU_TYPE_FORWARD =
   MenuButton* button_;  // Weak; comes from nib.
   scoped_ptr<BackForwardMenuModel> model_;
   scoped_nsobject<NSMenu> backForwardMenu_;
+  scoped_ptr<BackForwardMenuControllerInternal::MenuDelegate>
+      contextMenuDelegate_;
 }
 
 // Type (back or forwards); can only be set on initialization.
@@ -37,6 +44,7 @@ const BackForwardMenuType BACK_FORWARD_MENU_TYPE_FORWARD =
 - (id)initWithBrowser:(Browser*)browser
             modelType:(BackForwardMenuType)type
                button:(MenuButton*)button;
+- (void)onIconChanged:(int) modelIndex;

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

No space after the ).

 
 @end  // @interface BackForwardMenuController