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

Side by Side Diff: chrome/browser/notifications/message_center_notification_manager.h

Issue 1103713003: win: Move a few WeakPtrFactories to the end of their containing classes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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
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_NOTIFICATIONS_MESSAGE_CENTER_NOTIFICATION_MANAGER_H_ 5 #ifndef CHROME_BROWSER_NOTIFICATIONS_MESSAGE_CENTER_NOTIFICATION_MANAGER_H_
6 #define CHROME_BROWSER_NOTIFICATIONS_MESSAGE_CENTER_NOTIFICATION_MANAGER_H_ 6 #define CHROME_BROWSER_NOTIFICATIONS_MESSAGE_CENTER_NOTIFICATION_MANAGER_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 10
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 BooleanPrefMember first_run_pref_; 131 BooleanPrefMember first_run_pref_;
132 132
133 // The timer after which we will show the first run balloon. This timer is 133 // The timer after which we will show the first run balloon. This timer is
134 // restarted every time the message center is closed and every time the last 134 // restarted every time the message center is closed and every time the last
135 // popup disappears from the screen. 135 // popup disappears from the screen.
136 base::OneShotTimer<MessageCenterNotificationManager> first_run_balloon_timer_; 136 base::OneShotTimer<MessageCenterNotificationManager> first_run_balloon_timer_;
137 137
138 // The first-run balloon will be shown |first_run_idle_timeout_| after all 138 // The first-run balloon will be shown |first_run_idle_timeout_| after all
139 // popups go away and the user has notifications in the message center. 139 // popups go away and the user has notifications in the message center.
140 base::TimeDelta first_run_idle_timeout_; 140 base::TimeDelta first_run_idle_timeout_;
141
142 // Provides weak pointers for the purpose of the first run timer.
143 base::WeakPtrFactory<MessageCenterNotificationManager> weak_factory_;
144 #endif 141 #endif
145 142
146 scoped_ptr<message_center::NotifierSettingsProvider> settings_provider_; 143 scoped_ptr<message_center::NotifierSettingsProvider> settings_provider_;
147 144
148 // To own the blockers. 145 // To own the blockers.
149 ScopedVector<message_center::NotificationBlocker> blockers_; 146 ScopedVector<message_center::NotificationBlocker> blockers_;
150 147
151 NotificationSystemObserver system_observer_; 148 NotificationSystemObserver system_observer_;
152 149
153 // Keeps track of all notification statistics for UMA purposes. 150 // Keeps track of all notification statistics for UMA purposes.
154 MessageCenterStatsCollector stats_collector_; 151 MessageCenterStatsCollector stats_collector_;
155 152
156 // Keeps track of notifications specific to Google Now for UMA purposes. 153 // Keeps track of notifications specific to Google Now for UMA purposes.
157 GoogleNowNotificationStatsCollector google_now_stats_collector_; 154 GoogleNowNotificationStatsCollector google_now_stats_collector_;
158 155
156 #if defined(OS_WIN)
157 // Provides weak pointers for the purpose of the first run timer.
158 base::WeakPtrFactory<MessageCenterNotificationManager> weak_factory_;
159 #endif
160
159 DISALLOW_COPY_AND_ASSIGN(MessageCenterNotificationManager); 161 DISALLOW_COPY_AND_ASSIGN(MessageCenterNotificationManager);
160 }; 162 };
161 163
162 #endif // CHROME_BROWSER_NOTIFICATIONS_MESSAGE_CENTER_NOTIFICATION_MANAGER_H_ 164 #endif // CHROME_BROWSER_NOTIFICATIONS_MESSAGE_CENTER_NOTIFICATION_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698