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

Side by Side Diff: chrome/browser/ui/gtk/location_bar_view_gtk.h

Issue 10381105: Refactor UI "badge" (page action) logic into a BadgeController interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add missing NULL checks 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_ 5 #ifndef CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_
6 #define CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_ 6 #define CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_
7 #pragma once 7 #pragma once
8 8
9 #include <gtk/gtk.h> 9 #include <gtk/gtk.h>
10 10
(...skipping 235 matching lines...) Expand 10 before | Expand all | Expand 10 after
246 246
247 // Simulate left mouse click on the page action button. 247 // Simulate left mouse click on the page action button.
248 void TestActivatePageAction(); 248 void TestActivatePageAction();
249 249
250 // Implement the content::NotificationObserver interface. 250 // Implement the content::NotificationObserver interface.
251 virtual void Observe(int type, 251 virtual void Observe(int type,
252 const content::NotificationSource& source, 252 const content::NotificationSource& source,
253 const content::NotificationDetails& details) OVERRIDE; 253 const content::NotificationDetails& details) OVERRIDE;
254 254
255 private: 255 private:
256 // Show the popup for this page action. Returns true if a popup was shown.
257 bool ShowPopup();
258
259 // Connect the accelerator for the page action popup. 256 // Connect the accelerator for the page action popup.
260 void ConnectPageActionAccelerator(); 257 void ConnectPageActionAccelerator();
261 258
262 // Disconnect the accelerator for the page action popup. 259 // Disconnect the accelerator for the page action popup.
263 void DisconnectPageActionAccelerator(); 260 void DisconnectPageActionAccelerator();
264 261
265 CHROMEGTK_CALLBACK_1(PageActionViewGtk, gboolean, OnButtonPressed, 262 CHROMEGTK_CALLBACK_1(PageActionViewGtk, gboolean, OnButtonPressed,
266 GdkEventButton*); 263 GdkEventButton*);
267 CHROMEGTK_CALLBACK_1(PageActionViewGtk, gboolean, OnExposeEvent, 264 CHROMEGTK_CALLBACK_1(PageActionViewGtk, gboolean, OnExposeEvent,
268 GdkEventExpose*); 265 GdkEventExpose*);
(...skipping 224 matching lines...) Expand 10 before | Expand all | Expand 10 after
493 // The last search keyword that was shown via the |tab_to_search_box_|. 490 // The last search keyword that was shown via the |tab_to_search_box_|.
494 string16 last_keyword_; 491 string16 last_keyword_;
495 492
496 // Used to change the visibility of the star decoration. 493 // Used to change the visibility of the star decoration.
497 BooleanPrefMember edit_bookmarks_enabled_; 494 BooleanPrefMember edit_bookmarks_enabled_;
498 495
499 DISALLOW_COPY_AND_ASSIGN(LocationBarViewGtk); 496 DISALLOW_COPY_AND_ASSIGN(LocationBarViewGtk);
500 }; 497 };
501 498
502 #endif // CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_ 499 #endif // CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_
OLDNEW
« no previous file with comments | « chrome/browser/extensions/page_action_controller.cc ('k') | chrome/browser/ui/gtk/location_bar_view_gtk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698