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

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

Issue 9968076: Remove Inspect Popup command from browser actions. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Moved includes to cc Created 8 years, 8 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/browser_actions_toolbar_gtk.cc
diff --git a/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc b/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc
index 588def49b0e3ded74640f92b3e12a9c82d48101b..762418dabb1c3a27071a3c091e693485449f235d 100644
--- a/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc
+++ b/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc
@@ -89,7 +89,6 @@ using ui::SimpleMenuModel;
class BrowserActionButton : public content::NotificationObserver,
public ImageLoadingTracker::Observer,
- public ExtensionContextMenuModel::PopupDelegate,
public MenuGtk::Delegate {
public:
BrowserActionButton(BrowserActionsToolbarGtk* toolbar,
@@ -241,7 +240,7 @@ class BrowserActionButton : public content::NotificationObserver,
return NULL;
context_menu_model_ =
- new ExtensionContextMenuModel(extension_, toolbar_->browser(), this);
+ new ExtensionContextMenuModel(extension_, toolbar_->browser());
context_menu_.reset(
new MenuGtk(this, context_menu_model_.get()));
return context_menu_.get();
@@ -286,11 +285,6 @@ class BrowserActionButton : public content::NotificationObserver,
return false;
}
- // ExtensionContextMenuModel::PopupDelegate implementation.
- virtual void InspectPopup(ExtensionAction* action) {
- ShowPopup(true);
- }
-
void SetImage(GdkPixbuf* image) {
if (!image_) {
image_ = gtk_image_new_from_pixbuf(image);

Powered by Google App Engine
This is Rietveld 408576698