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

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

Issue 653933003: Resubmit: Rename BrowserActionView to ToolbarActionView (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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/toolbar/chevron_menu_button.cc
diff --git a/chrome/browser/ui/views/toolbar/chevron_menu_button.cc b/chrome/browser/ui/views/toolbar/chevron_menu_button.cc
index bdd15e1dd20292e29f40b16728883b16c8e13d1b..d1997f7cb79976f2234a43e3a895aa99826d0c4c 100644
--- a/chrome/browser/ui/views/toolbar/chevron_menu_button.cc
+++ b/chrome/browser/ui/views/toolbar/chevron_menu_button.cc
@@ -15,8 +15,8 @@
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/views/extensions/browser_action_drag_data.h"
#include "chrome/browser/ui/views/extensions/extension_action_view_controller.h"
-#include "chrome/browser/ui/views/toolbar/browser_action_view.h"
#include "chrome/browser/ui/views/toolbar/browser_actions_container.h"
+#include "chrome/browser/ui/views/toolbar/toolbar_action_view.h"
#include "extensions/common/extension.h"
#include "ui/views/border.h"
#include "ui/views/controls/button/label_button_border.h"
@@ -28,7 +28,7 @@
namespace {
// In the browser actions container's chevron menu, a menu item view's icon
-// comes from BrowserActionView::GetIconWithBadge() when the menu item view is
+// comes from ToolbarActionView::GetIconWithBadge() when the menu item view is
// created. But, the browser action's icon may not be loaded in time because it
// is read from file system in another thread.
// The IconUpdater will update the menu item view's icon when the browser
@@ -45,7 +45,7 @@ class IconUpdater : public ExtensionActionIconFactory::Observer {
}
~IconUpdater() override { view_controller_->set_icon_observer(NULL); }
- // BrowserActionView::IconObserver:
+ // ExtensionActionIconFactory::Observer:
void OnIconUpdated() override {
menu_item_view_->SetIcon(view_controller_->GetIconWithBadge());
}
@@ -123,7 +123,7 @@ class ChevronMenuButton::MenuController : public views::MenuDelegate {
// Resposible for running the menu.
scoped_ptr<views::MenuRunner> menu_runner_;
- // The index into the BrowserActionView vector, indicating where to start
+ // The index into the ToolbarActionView vector, indicating where to start
// picking browser actions to draw.
int start_index_;
@@ -155,9 +155,9 @@ ChevronMenuButton::MenuController::MenuController(
size_t command_id = 1; // Menu id 0 is reserved, start with 1.
for (size_t i = start_index_;
- i < browser_actions_container_->num_browser_actions(); ++i) {
- BrowserActionView* view =
- browser_actions_container_->GetBrowserActionViewAt(i);
+ i < browser_actions_container_->num_toolbar_actions(); ++i) {
+ ToolbarActionView* view =
+ browser_actions_container_->GetToolbarActionViewAt(i);
ExtensionActionViewController* view_controller =
static_cast<ExtensionActionViewController*>(view->view_controller());
views::MenuItemView* menu_item = menu_->AppendMenuItemWithIcon(
@@ -209,13 +209,13 @@ void ChevronMenuButton::MenuController::CloseMenu() {
}
bool ChevronMenuButton::MenuController::IsCommandEnabled(int id) const {
- BrowserActionView* view =
- browser_actions_container_->GetBrowserActionViewAt(start_index_ + id - 1);
+ ToolbarActionView* view =
+ browser_actions_container_->GetToolbarActionViewAt(start_index_ + id - 1);
return view->view_controller()->IsEnabled(view->GetCurrentWebContents());
}
void ChevronMenuButton::MenuController::ExecuteCommand(int id) {
- browser_actions_container_->GetBrowserActionViewAt(start_index_ + id - 1)->
+ browser_actions_container_->GetToolbarActionViewAt(start_index_ + id - 1)->
view_controller()->ExecuteAction(true);
}
@@ -224,7 +224,7 @@ bool ChevronMenuButton::MenuController::ShowContextMenu(
int id,
const gfx::Point& p,
ui::MenuSourceType source_type) {
- BrowserActionView* view = browser_actions_container_->GetBrowserActionViewAt(
+ ToolbarActionView* view = browser_actions_container_->GetToolbarActionViewAt(
start_index_ + id - 1);
ExtensionActionViewController* view_controller =
static_cast<ExtensionActionViewController*>(view->view_controller());

Powered by Google App Engine
This is Rietveld 408576698