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

Unified Diff: chrome/browser/ui/gtk/location_bar_view_gtk.cc

Issue 10446034: Rename ActionBoxController -> LocationBarController. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: grammar Created 8 years, 7 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/gtk/location_bar_view_gtk.cc
diff --git a/chrome/browser/ui/gtk/location_bar_view_gtk.cc b/chrome/browser/ui/gtk/location_bar_view_gtk.cc
index 7a13f8166f1951b1c7b8423502d1dc8132df4fe7..a432b4d63c887be9e595ae2ffe8d73caba3c0530 100644
--- a/chrome/browser/ui/gtk/location_bar_view_gtk.cc
+++ b/chrome/browser/ui/gtk/location_bar_view_gtk.cc
@@ -24,13 +24,13 @@
#include "chrome/browser/command_updater.h"
#include "chrome/browser/content_settings/tab_specific_content_settings.h"
#include "chrome/browser/defaults.h"
-#include "chrome/browser/extensions/action_box_controller.h"
#include "chrome/browser/extensions/api/commands/command_service.h"
#include "chrome/browser/extensions/api/commands/command_service_factory.h"
#include "chrome/browser/extensions/extension_browser_event_router.h"
#include "chrome/browser/extensions/extension_service.h"
#include "chrome/browser/extensions/extension_tab_helper.h"
#include "chrome/browser/extensions/extension_tab_util.h"
+#include "chrome/browser/extensions/location_bar_controller.h"
#include "chrome/browser/favicon/favicon_tab_helper.h"
#include "chrome/browser/instant/instant_controller.h"
#include "chrome/browser/profiles/profile.h"
@@ -84,7 +84,7 @@
using content::NavigationEntry;
using content::OpenURLParams;
using content::WebContents;
-using extensions::ActionBoxController;
+using extensions::LocationBarController;
using extensions::Extension;
namespace {
@@ -694,8 +694,8 @@ void LocationBarViewGtk::UpdatePageActions() {
TabContentsWrapper* tab_contents = GetTabContentsWrapper();
if (tab_contents) {
- ActionBoxController* controller =
- tab_contents->extension_tab_helper()->action_box_controller();
+ LocationBarController* controller =
+ tab_contents->extension_tab_helper()->location_bar_controller();
page_actions.swap(*controller->GetCurrentActions());
}
@@ -1764,21 +1764,21 @@ gboolean LocationBarViewGtk::PageActionViewGtk::OnButtonPressed(
if (!extension)
return TRUE;
- ActionBoxController* controller =
- tab_contents->extension_tab_helper()->action_box_controller();
+ LocationBarController* controller =
+ tab_contents->extension_tab_helper()->location_bar_controller();
switch (controller->OnClicked(extension->id(), event->button)) {
- case ActionBoxController::ACTION_NONE:
+ case LocationBarController::ACTION_NONE:
break;
- case ActionBoxController::ACTION_SHOW_POPUP:
+ case LocationBarController::ACTION_SHOW_POPUP:
ExtensionPopupGtk::Show(
page_action_->GetPopupUrl(current_tab_id_),
owner_->browser_,
event_box_.get());
break;
- case ActionBoxController::ACTION_SHOW_CONTEXT_MENU:
+ case LocationBarController::ACTION_SHOW_CONTEXT_MENU:
context_menu_model_ =
new ExtensionContextMenuModel(extension, owner_->browser_);
context_menu_.reset(

Powered by Google App Engine
This is Rietveld 408576698