Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(58)

Unified Diff: chrome/browser/ui/views/toolbar/browser_actions_container.cc

Issue 390183002: MacViews: Move menu run types parameter to constructor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/toolbar/browser_actions_container.cc
diff --git a/chrome/browser/ui/views/toolbar/browser_actions_container.cc b/chrome/browser/ui/views/toolbar/browser_actions_container.cc
index 05234b1e0253685c1c493999e53d5f1a206d0a2e..df22b9b3287323a545827a4cc5ff7cdd3e768826 100644
--- a/chrome/browser/ui/views/toolbar/browser_actions_container.cc
+++ b/chrome/browser/ui/views/toolbar/browser_actions_container.cc
@@ -511,11 +511,15 @@ void BrowserActionsContainer::GetAccessibleState(
void BrowserActionsContainer::OnMenuButtonClicked(views::View* source,
const gfx::Point& point) {
if (source == chevron_) {
- overflow_menu_ = new BrowserActionOverflowMenuController(
- this, browser_, chevron_, browser_action_views_,
- VisibleBrowserActions());
+ overflow_menu_ =
+ new BrowserActionOverflowMenuController(this,
+ browser_,
+ chevron_,
+ browser_action_views_,
+ VisibleBrowserActions(),
+ false);
overflow_menu_->set_observer(this);
- overflow_menu_->RunMenu(GetWidget(), false);
+ overflow_menu_->RunMenu(GetWidget());
}
}
@@ -933,10 +937,15 @@ void BrowserActionsContainer::StartShowFolderDropMenuTimer() {
void BrowserActionsContainer::ShowDropFolder() {
DCHECK(!overflow_menu_);
SetDropIndicator(-1);
- overflow_menu_ = new BrowserActionOverflowMenuController(
- this, browser_, chevron_, browser_action_views_, VisibleBrowserActions());
+ overflow_menu_ =
+ new BrowserActionOverflowMenuController(this,
+ browser_,
+ chevron_,
+ browser_action_views_,
+ VisibleBrowserActions(),
+ true);
overflow_menu_->set_observer(this);
- overflow_menu_->RunMenu(GetWidget(), true);
+ overflow_menu_->RunMenu(GetWidget());
}
void BrowserActionsContainer::SetDropIndicator(int x_pos) {
« no previous file with comments | « chrome/browser/ui/views/toolbar/browser_action_view.cc ('k') | chrome/browser/ui/views/toolbar/toolbar_button.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698