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

Side by Side Diff: chrome/browser/ui/views/status_icons/status_tray_win.h

Issue 407653002: Give StatusTrayStateChangerProxy a virtual destructor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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_VIEWS_STATUS_ICONS_STATUS_TRAY_WIN_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_STATUS_ICONS_STATUS_TRAY_WIN_H_
6 #define CHROME_BROWSER_UI_VIEWS_STATUS_ICONS_STATUS_TRAY_WIN_H_ 6 #define CHROME_BROWSER_UI_VIEWS_STATUS_ICONS_STATUS_TRAY_WIN_H_
7 7
8 #include <windows.h> 8 #include <windows.h>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
11 #include "base/memory/scoped_ptr.h" 11 #include "base/memory/scoped_ptr.h"
12 #include "chrome/browser/status_icons/status_tray.h" 12 #include "chrome/browser/status_icons/status_tray.h"
13 13
14 class StatusIconWin; 14 class StatusIconWin;
15 15
16 // A class that's responsible for increasing, if possible, the visibility 16 // A class that's responsible for increasing, if possible, the visibility
17 // of a status tray icon on the taskbar. The default implementation sends 17 // of a status tray icon on the taskbar. The default implementation sends
18 // a task to a worker thread each time EnqueueChange is called. 18 // a task to a worker thread each time EnqueueChange is called.
19 class StatusTrayStateChangerProxy { 19 class StatusTrayStateChangerProxy {
20 public: 20 public:
21 virtual ~StatusTrayStateChangerProxy();
22
21 // Called by StatusTrayWin to request upgraded visibility on the icon 23 // Called by StatusTrayWin to request upgraded visibility on the icon
22 // represented by the |icon_id|, |window| pair. 24 // represented by the |icon_id|, |window| pair.
23 virtual void EnqueueChange(UINT icon_id, HWND window) = 0; 25 virtual void EnqueueChange(UINT icon_id, HWND window) = 0;
24 }; 26 };
25 27
26 class StatusTrayWin : public StatusTray { 28 class StatusTrayWin : public StatusTray {
27 public: 29 public:
28 StatusTrayWin(); 30 StatusTrayWin();
29 ~StatusTrayWin(); 31 ~StatusTrayWin();
30 32
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 73
72 // Manages changes performed on a background thread to manipulate visibility 74 // Manages changes performed on a background thread to manipulate visibility
73 // of notification icons. 75 // of notification icons.
74 scoped_ptr<StatusTrayStateChangerProxy> state_changer_proxy_; 76 scoped_ptr<StatusTrayStateChangerProxy> state_changer_proxy_;
75 77
76 DISALLOW_COPY_AND_ASSIGN(StatusTrayWin); 78 DISALLOW_COPY_AND_ASSIGN(StatusTrayWin);
77 }; 79 };
78 80
79 #endif // CHROME_BROWSER_UI_VIEWS_STATUS_ICONS_STATUS_TRAY_WIN_H_ 81 #endif // CHROME_BROWSER_UI_VIEWS_STATUS_ICONS_STATUS_TRAY_WIN_H_
80 82
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698