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

Side by Side Diff: chrome/browser/ui/panels/display_settings_provider.h

Issue 12086018: GTTF: Add missing virtual destructors. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixes Created 7 years, 10 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_PANELS_DISPLAY_SETTINGS_PROVIDER_H_ 5 #ifndef CHROME_BROWSER_UI_PANELS_DISPLAY_SETTINGS_PROVIDER_H_
6 #define CHROME_BROWSER_UI_PANELS_DISPLAY_SETTINGS_PROVIDER_H_ 6 #define CHROME_BROWSER_UI_PANELS_DISPLAY_SETTINGS_PROVIDER_H_
7 7
8 #include "base/observer_list.h" 8 #include "base/observer_list.h"
9 #include "base/timer.h" 9 #include "base/timer.h"
10 #include "ui/gfx/rect.h" 10 #include "ui/gfx/rect.h"
(...skipping 15 matching lines...) Expand all
26 // Indicates current visibility state of the desktop bar. 26 // Indicates current visibility state of the desktop bar.
27 enum DesktopBarVisibility { 27 enum DesktopBarVisibility {
28 DESKTOP_BAR_VISIBLE, 28 DESKTOP_BAR_VISIBLE,
29 DESKTOP_BAR_ANIMATING, 29 DESKTOP_BAR_ANIMATING,
30 DESKTOP_BAR_HIDDEN 30 DESKTOP_BAR_HIDDEN
31 }; 31 };
32 32
33 class DisplayAreaObserver { 33 class DisplayAreaObserver {
34 public: 34 public:
35 virtual void OnDisplayAreaChanged(const gfx::Rect& display_area) = 0; 35 virtual void OnDisplayAreaChanged(const gfx::Rect& display_area) = 0;
36
37 protected:
38 virtual ~DisplayAreaObserver() { }
36 }; 39 };
37 40
38 class DesktopBarObserver { 41 class DesktopBarObserver {
39 public: 42 public:
40 virtual void OnAutoHidingDesktopBarVisibilityChanged( 43 virtual void OnAutoHidingDesktopBarVisibilityChanged(
41 DesktopBarAlignment alignment, DesktopBarVisibility visibility) = 0; 44 DesktopBarAlignment alignment, DesktopBarVisibility visibility) = 0;
45
46 protected:
47 virtual ~DesktopBarObserver() { }
42 }; 48 };
43 49
44 class FullScreenObserver { 50 class FullScreenObserver {
45 public: 51 public:
46 virtual void OnFullScreenModeChanged(bool is_full_screen) = 0; 52 virtual void OnFullScreenModeChanged(bool is_full_screen) = 0;
53
54 protected:
55 virtual ~FullScreenObserver() { }
47 }; 56 };
48 57
49 static DisplaySettingsProvider* Create(); 58 static DisplaySettingsProvider* Create();
50 59
51 virtual ~DisplaySettingsProvider(); 60 virtual ~DisplaySettingsProvider();
52 61
53 // Subscribes/unsubscribes from the display settings change notification. 62 // Subscribes/unsubscribes from the display settings change notification.
54 void AddDisplayAreaObserver(DisplayAreaObserver* observer); 63 void AddDisplayAreaObserver(DisplayAreaObserver* observer);
55 void RemoveDisplayAreaObserver(DisplayAreaObserver* observer); 64 void RemoveDisplayAreaObserver(DisplayAreaObserver* observer);
56 65
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
157 // True if full screen mode or presentation mode is entered. 166 // True if full screen mode or presentation mode is entered.
158 bool is_full_screen_; 167 bool is_full_screen_;
159 168
160 // Timer used to detect full-screen mode change. 169 // Timer used to detect full-screen mode change.
161 base::RepeatingTimer<DisplaySettingsProvider> full_screen_mode_timer_; 170 base::RepeatingTimer<DisplaySettingsProvider> full_screen_mode_timer_;
162 171
163 DISALLOW_COPY_AND_ASSIGN(DisplaySettingsProvider); 172 DISALLOW_COPY_AND_ASSIGN(DisplaySettingsProvider);
164 }; 173 };
165 174
166 #endif // CHROME_BROWSER_UI_PANELS_DISPLAY_SETTINGS_PROVIDER_H_ 175 #endif // CHROME_BROWSER_UI_PANELS_DISPLAY_SETTINGS_PROVIDER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698