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

Side by Side Diff: chrome/browser/gtk/overflow_button.h

Issue 5648004: Add the "virtual" keyword on method overrides that are missing it. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Missing file Created 10 years 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) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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_GTK_OVERFLOW_BUTTON_H_ 5 #ifndef CHROME_BROWSER_GTK_OVERFLOW_BUTTON_H_
6 #define CHROME_BROWSER_GTK_OVERFLOW_BUTTON_H_ 6 #define CHROME_BROWSER_GTK_OVERFLOW_BUTTON_H_
7 #pragma once 7 #pragma once
8 8
9 #include "chrome/browser/gtk/owned_widget_gtk.h" 9 #include "chrome/browser/gtk/owned_widget_gtk.h"
10 #include "chrome/common/notification_observer.h" 10 #include "chrome/common/notification_observer.h"
11 #include "chrome/common/notification_registrar.h" 11 #include "chrome/common/notification_registrar.h"
12 12
13 typedef struct _GtkWidget GtkWidget; 13 typedef struct _GtkWidget GtkWidget;
14 class Profile; 14 class Profile;
15 15
16 // An overflow chevron button. The button itself is a plain gtk_chrome_button, 16 // An overflow chevron button. The button itself is a plain gtk_chrome_button,
17 // and this class handles theming it. 17 // and this class handles theming it.
18 class OverflowButton : public NotificationObserver { 18 class OverflowButton : public NotificationObserver {
19 public: 19 public:
20 explicit OverflowButton(Profile* profile); 20 explicit OverflowButton(Profile* profile);
21 virtual ~OverflowButton(); 21 virtual ~OverflowButton();
22 22
23 GtkWidget* widget() { return widget_.get(); } 23 GtkWidget* widget() { return widget_.get(); }
24 24
25 private: 25 private:
26 // NotificationObserver implementation. 26 // NotificationObserver implementation.
27 void Observe(NotificationType type, 27 virtual void Observe(NotificationType type,
28 const NotificationSource& source, 28 const NotificationSource& source,
29 const NotificationDetails& details); 29 const NotificationDetails& details);
30 30
31 OwnedWidgetGtk widget_; 31 OwnedWidgetGtk widget_;
32 32
33 Profile* profile_; 33 Profile* profile_;
34 34
35 NotificationRegistrar registrar_; 35 NotificationRegistrar registrar_;
36 }; 36 };
37 37
38 #endif // CHROME_BROWSER_GTK_OVERFLOW_BUTTON_H_ 38 #endif // CHROME_BROWSER_GTK_OVERFLOW_BUTTON_H_
OLDNEW
« no previous file with comments | « chrome/browser/gtk/options/passwords_page_gtk.h ('k') | chrome/browser/gtk/slide_animator_gtk.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698