Moves ownership of MenuItemView to MenuRunner as well as responbility

chrome/browser/ui/views/extensions/browser_action_overflow_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_VIEWS_EXTENSIONS_BROWSER_ACTION_OVERFLOW_MENU_CONTROLLER_H_
 #define CHROME_BROWSER_UI_VIEWS_EXTENSIONS_BROWSER_ACTION_OVERFLOW_MENU_CONTROLLER_H_
 #pragma once
 
 #include <set>
 #include <vector>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/task.h"
 #include "views/controls/menu/menu_delegate.h"
 
 class BrowserActionsContainer;
 class BrowserActionView;
 
 namespace views {
+class MenuRunner;
 class Widget;
 }
 
 // This class handles the overflow menu for browser actions (showing the menu,
 // drag and drop, etc). This class manages its own lifetime.
 class BrowserActionOverflowMenuController : public views::MenuDelegate {
  public:
   // The observer is notified prior to the menu being deleted.
   class Observer {
    public:
@@ skipping 53 matching lines @@
 
   // A pointer to the browser action container that owns the overflow menu.
   BrowserActionsContainer* owner_;
 
   // The observer, may be null.
   Observer* observer_;
 
   // A pointer to the overflow menu button that we are showing the menu for.
   views::MenuButton* menu_button_;
 
-  // The overflow menu for the menu button.
-  scoped_ptr<views::MenuItemView> menu_;
+  // The overflow menu for the menu button. Owned by |menu_runner_|.
+  views::MenuItemView* menu_;
+
+  // Resposible for running the menu.
+  scoped_ptr<views::MenuRunner> menu_runner_;
 
   // The views vector of all the browser actions the container knows about. We
   // won't show all items, just the one starting at |start_index| and above.
   const std::vector<BrowserActionView*>* views_;
 
   // The index into the BrowserActionView vector, indicating where to start
   // picking browser actions to draw.
   int start_index_;
 
   // Whether this controller is being used for drop.
   bool for_drop_;
 
   friend class DeleteTask<BrowserActionOverflowMenuController>;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserActionOverflowMenuController);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_EXTENSIONS_BROWSER_ACTION_OVERFLOW_MENU_CONTROLLER_H_