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

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

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_COCOA_TOOLBAR_BACK_FORWARD_MENU_CONTROLLER_H_
 #define CHROME_BROWSER_UI_COCOA_TOOLBAR_BACK_FORWARD_MENU_CONTROLLER_H_
 #pragma once
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/scoped_nsobject.h"
 #include "base/scoped_ptr.h"
 #include "chrome/browser/ui/toolbar/back_forward_menu_model.h"
 
 @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;
 const BackForwardMenuType BACK_FORWARD_MENU_TYPE_FORWARD =
     BackForwardMenuModel::FORWARD_MENU;
 
 // A class that manages the back/forward menu (and delayed-menu button, and
 // model).
 
 @interface BackForwardMenuController : NSObject {
  @private
   BackForwardMenuType type_;
   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.
 @property(readonly, nonatomic) BackForwardMenuType type;
 
 - (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
 
 #endif  // CHROME_BROWSER_UI_COCOA_TOOLBAR_BACK_FORWARD_MENU_CONTROLLER_H_