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

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

Issue 10677009: Move command handling and updating off Browser and onto a helper object. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 6 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 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 } 53 }
54 54
55 namespace ui { 55 namespace ui {
56 class AcceleratorGtk; 56 class AcceleratorGtk;
57 } 57 }
58 58
59 class LocationBarViewGtk : public OmniboxEditController, 59 class LocationBarViewGtk : public OmniboxEditController,
60 public LocationBar, 60 public LocationBar,
61 public LocationBarTesting, 61 public LocationBarTesting,
62 public content::NotificationObserver, 62 public content::NotificationObserver,
63 public CommandUpdater::CommandObserver { 63 public CommandObserver {
64 public: 64 public:
65 explicit LocationBarViewGtk(Browser* browser); 65 explicit LocationBarViewGtk(Browser* browser);
66 virtual ~LocationBarViewGtk(); 66 virtual ~LocationBarViewGtk();
67 67
68 void Init(bool popup_window_mode); 68 void Init(bool popup_window_mode);
69 69
70 // Returns the widget the caller should host. You must call Init() first. 70 // Returns the widget the caller should host. You must call Init() first.
71 GtkWidget* widget() { return hbox_.get(); } 71 GtkWidget* widget() { return hbox_.get(); }
72 72
73 // Returns the widget the page info bubble should point to. 73 // Returns the widget the page info bubble should point to.
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 virtual int PageActionVisibleCount() OVERRIDE; 155 virtual int PageActionVisibleCount() OVERRIDE;
156 virtual ExtensionAction* GetPageAction(size_t index) OVERRIDE; 156 virtual ExtensionAction* GetPageAction(size_t index) OVERRIDE;
157 virtual ExtensionAction* GetVisiblePageAction(size_t index) OVERRIDE; 157 virtual ExtensionAction* GetVisiblePageAction(size_t index) OVERRIDE;
158 virtual void TestPageActionPressed(size_t index) OVERRIDE; 158 virtual void TestPageActionPressed(size_t index) OVERRIDE;
159 159
160 // content::NotificationObserver: 160 // content::NotificationObserver:
161 virtual void Observe(int type, 161 virtual void Observe(int type,
162 const content::NotificationSource& source, 162 const content::NotificationSource& source,
163 const content::NotificationDetails& details) OVERRIDE; 163 const content::NotificationDetails& details) OVERRIDE;
164 164
165 // CommandUpdater::CommandObserver: 165 // CommandObserver:
166 virtual void EnabledStateChangedForCommand(int id, bool enabled) OVERRIDE; 166 virtual void EnabledStateChangedForCommand(int id, bool enabled) OVERRIDE;
167 167
168 // Edit background color. 168 // Edit background color.
169 static const GdkColor kBackgroundColor; 169 static const GdkColor kBackgroundColor;
170 170
171 private: 171 private:
172 class ContentSettingImageViewGtk : public BubbleDelegateGtk, 172 class ContentSettingImageViewGtk : public BubbleDelegateGtk,
173 public ui::AnimationDelegate { 173 public ui::AnimationDelegate {
174 public: 174 public:
175 ContentSettingImageViewGtk(ContentSettingsType content_type, 175 ContentSettingImageViewGtk(ContentSettingsType content_type,
(...skipping 349 matching lines...) Expand 10 before | Expand all | Expand 10 after
525 // The last search keyword that was shown via the |tab_to_search_box_|. 525 // The last search keyword that was shown via the |tab_to_search_box_|.
526 string16 last_keyword_; 526 string16 last_keyword_;
527 527
528 // Used to change the visibility of the star decoration. 528 // Used to change the visibility of the star decoration.
529 BooleanPrefMember edit_bookmarks_enabled_; 529 BooleanPrefMember edit_bookmarks_enabled_;
530 530
531 DISALLOW_COPY_AND_ASSIGN(LocationBarViewGtk); 531 DISALLOW_COPY_AND_ASSIGN(LocationBarViewGtk);
532 }; 532 };
533 533
534 #endif // CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_ 534 #endif // CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698