Add a menu to the Action Box Button.

chrome/browser/ui/cocoa/location_bar/plus_decoration.h

 // Copyright (c) 2012 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_LOCATION_BAR_PLUS_DECORATION_H_
 #define CHROME_BROWSER_UI_COCOA_LOCATION_BAR_PLUS_DECORATION_H_
 
 #import <Cocoa/Cocoa.h>
 
+#include "base/memory/scoped_ptr.h"
 #include "chrome/browser/ui/cocoa/location_bar/image_decoration.h"
 
+class Browser;
 class CommandUpdater;
+class LocationBarViewMac;
+@class MenuController;
+
+namespace ui {
+class SimpleMenuModel;
+}  // namespace ui
 
 // Note: this file is under development (see crbug.com/138118).
 
 // Plus icon on the right side of the location bar.
 class PlusDecoration : public ImageDecoration {
  public:
-  explicit PlusDecoration(CommandUpdater* command_updater);
+  PlusDecoration(LocationBarViewMac* owner,
+      CommandUpdater* command_updater,
+      Browser* browser);

[Scott Hess - ex-Googler, 2012/08/08 15:00:14]

Double-check the style guide as to whether continuation lines should line up
indented further (like L, C, and B in a vertical line).

[beaudoin, 2012/08/08 21:38:08]

Done.

   virtual ~PlusDecoration();
 
   // Implement |LocationBarDecoration|.
   virtual bool AcceptsMousePress() OVERRIDE;
   virtual bool OnMousePressed(NSRect frame) OVERRIDE;
   virtual NSString* GetToolTip() OVERRIDE;
 
  private:
+  // Creates a menu model and a menu controller.
+  void CreateMenuController();
+
+  // Owner of the decoration, used to obtain the menu.
+  LocationBarViewMac* owner_;
+
   CommandUpdater* command_updater_;  // Weak, owned by Browser.
 
+  Browser* browser_;
+
   // The string to show for a tooltip.
   scoped_nsobject<NSString> tooltip_;
 
+  // Model for the menu attached to the plus decoration.
+  scoped_ptr<ui::SimpleMenuModel> menu_model_;
+
+  // Controller for the menu attached to the plus decoration.
+  scoped_nsobject<MenuController> menuController_;

[Scott Hess - ex-Googler, 2012/08/08 15:00:14]

Studly caps is for Objective-C files.  Admittedly ambiguous at this point, but
I'd lean towards c++ style.

[beaudoin, 2012/08/08 21:38:08]

Done.

+
+  // Popup cell.

[Scott Hess - ex-Googler, 2012/08/08 15:00:14]

Redundant comments document things multiple times!

[beaudoin, 2012/08/08 21:38:08]

Done.

+  scoped_nsobject<NSPopUpButtonCell> popUpCell_;
+
   DISALLOW_COPY_AND_ASSIGN(PlusDecoration);
 };
 
 #endif  // CHROME_BROWSER_UI_COCOA_LOCATION_BAR_PLUS_DECORATION_H_