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

Side by Side Diff: chrome/browser/ui/views/toolbar/chevron_menu_button.cc

Issue 670463004: Make a platform-independent ToolbarActionViewController (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/toolbar/chevron_menu_button.h" 5 #include "chrome/browser/ui/views/toolbar/chevron_menu_button.h"
6 6
7 #include "base/memory/scoped_vector.h" 7 #include "base/memory/scoped_vector.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "chrome/browser/extensions/extension_action.h" 10 #include "chrome/browser/extensions/extension_action.h"
11 #include "chrome/browser/extensions/extension_action_icon_factory.h" 11 #include "chrome/browser/extensions/extension_action_icon_factory.h"
12 #include "chrome/browser/extensions/extension_context_menu_model.h" 12 #include "chrome/browser/extensions/extension_context_menu_model.h"
13 #include "chrome/browser/extensions/extension_toolbar_model.h" 13 #include "chrome/browser/extensions/extension_toolbar_model.h"
14 #include "chrome/browser/profiles/profile.h" 14 #include "chrome/browser/profiles/profile.h"
15 #include "chrome/browser/ui/browser.h" 15 #include "chrome/browser/ui/browser.h"
16 #include "chrome/browser/ui/views/extensions/browser_action_drag_data.h" 16 #include "chrome/browser/ui/views/extensions/browser_action_drag_data.h"
17 #include "chrome/browser/ui/views/extensions/extension_action_view_controller.h" 17 #include "chrome/browser/ui/views/extensions/extension_action_view_controller_vi ews.h"
18 #include "chrome/browser/ui/views/toolbar/browser_action_view.h" 18 #include "chrome/browser/ui/views/toolbar/browser_action_view.h"
19 #include "chrome/browser/ui/views/toolbar/browser_actions_container.h" 19 #include "chrome/browser/ui/views/toolbar/browser_actions_container.h"
20 #include "extensions/common/extension.h" 20 #include "extensions/common/extension.h"
21 #include "ui/views/border.h" 21 #include "ui/views/border.h"
22 #include "ui/views/controls/button/label_button_border.h" 22 #include "ui/views/controls/button/label_button_border.h"
23 #include "ui/views/controls/menu/menu_delegate.h" 23 #include "ui/views/controls/menu/menu_delegate.h"
24 #include "ui/views/controls/menu/menu_item_view.h" 24 #include "ui/views/controls/menu/menu_item_view.h"
25 #include "ui/views/controls/menu/menu_runner.h" 25 #include "ui/views/controls/menu/menu_runner.h"
26 #include "ui/views/metrics.h" 26 #include "ui/views/metrics.h"
27 27
28 namespace { 28 namespace {
29 29
30 // In the browser actions container's chevron menu, a menu item view's icon 30 // In the browser actions container's chevron menu, a menu item view's icon
31 // comes from BrowserActionView::GetIconWithBadge() when the menu item view is 31 // comes from BrowserActionView::GetIconWithBadge() when the menu item view is
32 // created. But, the browser action's icon may not be loaded in time because it 32 // created. But, the browser action's icon may not be loaded in time because it
33 // is read from file system in another thread. 33 // is read from file system in another thread.
34 // The IconUpdater will update the menu item view's icon when the browser 34 // The IconUpdater will update the menu item view's icon when the browser
35 // action's icon has been updated. 35 // action's icon has been updated.
36 class IconUpdater : public ExtensionActionIconFactory::Observer { 36 class IconUpdater : public ExtensionActionIconFactory::Observer {
37 public: 37 public:
38 IconUpdater(views::MenuItemView* menu_item_view, 38 IconUpdater(views::MenuItemView* menu_item_view,
39 ExtensionActionViewController* view_controller) 39 ExtensionActionViewControllerViews* view_controller)
40 : menu_item_view_(menu_item_view), 40 : menu_item_view_(menu_item_view),
41 view_controller_(view_controller) { 41 view_controller_(view_controller) {
42 DCHECK(menu_item_view); 42 DCHECK(menu_item_view);
43 DCHECK(view_controller); 43 DCHECK(view_controller);
44 view_controller->set_icon_observer(this); 44 view_controller->set_icon_observer(this);
45 } 45 }
46 virtual ~IconUpdater() { 46 virtual ~IconUpdater() {
47 view_controller_->set_icon_observer(NULL); 47 view_controller_->set_icon_observer(NULL);
48 } 48 }
49 49
50 // BrowserActionView::IconObserver: 50 // BrowserActionView::IconObserver:
51 virtual void OnIconUpdated() override { 51 virtual void OnIconUpdated() override {
52 menu_item_view_->SetIcon(view_controller_->GetIconWithBadge()); 52 menu_item_view_->SetIcon(view_controller_->GetIconWithBadge());
53 } 53 }
54 54
55 private: 55 private:
56 // The menu item view whose icon might be updated. 56 // The menu item view whose icon might be updated.
57 views::MenuItemView* menu_item_view_; 57 views::MenuItemView* menu_item_view_;
58 58
59 // The view controller to be observed. When its icon changes, update the 59 // The view controller to be observed. When its icon changes, update the
60 // corresponding menu item view's icon. 60 // corresponding menu item view's icon.
61 ExtensionActionViewController* view_controller_; 61 ExtensionActionViewControllerViews* view_controller_;
62 62
63 DISALLOW_COPY_AND_ASSIGN(IconUpdater); 63 DISALLOW_COPY_AND_ASSIGN(IconUpdater);
64 }; 64 };
65 65
66 } // namespace 66 } // namespace
67 67
68 // This class handles the overflow menu for browser actions. 68 // This class handles the overflow menu for browser actions.
69 class ChevronMenuButton::MenuController : public views::MenuDelegate { 69 class ChevronMenuButton::MenuController : public views::MenuDelegate {
70 public: 70 public:
71 MenuController(ChevronMenuButton* owner, 71 MenuController(ChevronMenuButton* owner,
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 menu_ = new views::MenuItemView(this); 153 menu_ = new views::MenuItemView(this);
154 menu_runner_.reset(new views::MenuRunner( 154 menu_runner_.reset(new views::MenuRunner(
155 menu_, for_drop_ ? views::MenuRunner::FOR_DROP : 0)); 155 menu_, for_drop_ ? views::MenuRunner::FOR_DROP : 0));
156 menu_->set_has_icons(true); 156 menu_->set_has_icons(true);
157 157
158 size_t command_id = 1; // Menu id 0 is reserved, start with 1. 158 size_t command_id = 1; // Menu id 0 is reserved, start with 1.
159 for (size_t i = start_index_; 159 for (size_t i = start_index_;
160 i < browser_actions_container_->num_browser_actions(); ++i) { 160 i < browser_actions_container_->num_browser_actions(); ++i) {
161 BrowserActionView* view = 161 BrowserActionView* view =
162 browser_actions_container_->GetBrowserActionViewAt(i); 162 browser_actions_container_->GetBrowserActionViewAt(i);
163 ExtensionActionViewController* view_controller =
164 static_cast<ExtensionActionViewController*>(view->view_controller());
165 views::MenuItemView* menu_item = menu_->AppendMenuItemWithIcon( 163 views::MenuItemView* menu_item = menu_->AppendMenuItemWithIcon(
166 command_id, 164 command_id,
167 base::UTF8ToUTF16(view_controller->extension()->name()), 165 view->view_controller()->GetActionName(),
168 view_controller->GetIconWithBadge()); 166 view->view_controller()->GetIconWithBadge());
169 167
170 // Set the tooltip for this item. 168 // Set the tooltip for this item.
171 menu_->SetTooltip( 169 menu_->SetTooltip(
172 view_controller->GetTooltip(view->GetCurrentWebContents()), 170 view->view_controller()->GetTooltip(view->GetCurrentWebContents()),
173 command_id); 171 command_id);
174 172
175 icon_updaters_.push_back(new IconUpdater(menu_item, view_controller)); 173 icon_updaters_.push_back(new IconUpdater(
174 menu_item,
175 static_cast<ExtensionActionViewControllerViews*>(
176 view->view_controller())));
176 177
177 ++command_id; 178 ++command_id;
178 } 179 }
179 } 180 }
180 181
181 ChevronMenuButton::MenuController::~MenuController() { 182 ChevronMenuButton::MenuController::~MenuController() {
182 } 183 }
183 184
184 void ChevronMenuButton::MenuController::RunMenu(views::Widget* window) { 185 void ChevronMenuButton::MenuController::RunMenu(views::Widget* window) {
185 gfx::Rect bounds = owner_->bounds(); 186 gfx::Rect bounds = owner_->bounds();
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 view_controller()->ExecuteAction(true); 222 view_controller()->ExecuteAction(true);
222 } 223 }
223 224
224 bool ChevronMenuButton::MenuController::ShowContextMenu( 225 bool ChevronMenuButton::MenuController::ShowContextMenu(
225 views::MenuItemView* source, 226 views::MenuItemView* source,
226 int id, 227 int id,
227 const gfx::Point& p, 228 const gfx::Point& p,
228 ui::MenuSourceType source_type) { 229 ui::MenuSourceType source_type) {
229 BrowserActionView* view = browser_actions_container_->GetBrowserActionViewAt( 230 BrowserActionView* view = browser_actions_container_->GetBrowserActionViewAt(
230 start_index_ + id - 1); 231 start_index_ + id - 1);
231 ExtensionActionViewController* view_controller = 232 ExtensionActionViewControllerViews* view_controller =
232 static_cast<ExtensionActionViewController*>(view->view_controller()); 233 static_cast<ExtensionActionViewControllerViews*>(view->view_controller());
233 if (!view_controller->extension()->ShowConfigureContextMenus()) 234 if (!view_controller->extension()->ShowConfigureContextMenus())
234 return false; 235 return false;
235 236
236 scoped_refptr<ExtensionContextMenuModel> context_menu_contents = 237 scoped_refptr<ExtensionContextMenuModel> context_menu_contents =
237 new ExtensionContextMenuModel(view_controller->extension(), 238 new ExtensionContextMenuModel(view_controller->extension(),
238 view->browser(), 239 view->browser(),
239 view_controller); 240 view_controller);
240 views::MenuRunner context_menu_runner(context_menu_contents.get(), 241 views::MenuRunner context_menu_runner(context_menu_contents.get(),
241 views::MenuRunner::HAS_MNEMONICS | 242 views::MenuRunner::HAS_MNEMONICS |
242 views::MenuRunner::IS_NESTED | 243 views::MenuRunner::IS_NESTED |
(...skipping 187 matching lines...) Expand 10 before | Expand all | Expand 10 after
430 void ChevronMenuButton::ShowOverflowMenu(bool for_drop) { 431 void ChevronMenuButton::ShowOverflowMenu(bool for_drop) {
431 DCHECK(!menu_controller_); 432 DCHECK(!menu_controller_);
432 menu_controller_.reset(new MenuController( 433 menu_controller_.reset(new MenuController(
433 this, browser_actions_container_, for_drop)); 434 this, browser_actions_container_, for_drop));
434 menu_controller_->RunMenu(GetWidget()); 435 menu_controller_->RunMenu(GetWidget());
435 } 436 }
436 437
437 void ChevronMenuButton::MenuDone() { 438 void ChevronMenuButton::MenuDone() {
438 menu_controller_.reset(); 439 menu_controller_.reset();
439 } 440 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698