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

Unified Diff: chrome/browser/background_mode_manager.h

Issue 3304015: Use PrefChangeRegistrar everywhere (Closed)
Patch Set: final version for commit Created 10 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/background_mode_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/background_mode_manager.h
diff --git a/chrome/browser/background_mode_manager.h b/chrome/browser/background_mode_manager.h
index 46e8847cb0dc160209fc2aded58e45a4289354bc..52f37da0bf8c70fc89158e20ed3fac33a2d9df6f 100644
--- a/chrome/browser/background_mode_manager.h
+++ b/chrome/browser/background_mode_manager.h
@@ -8,6 +8,7 @@
#include "app/menus/simple_menu_model.h"
#include "base/gtest_prod_util.h"
+#include "chrome/browser/prefs/pref_change_registrar.h"
#include "chrome/browser/status_icons/status_icon.h"
#include "chrome/common/notification_observer.h"
#include "chrome/common/notification_registrar.h"
@@ -142,6 +143,9 @@ class BackgroundModeManager
// Reference to our status icon (if any) - owned by the StatusTray.
StatusIcon* status_icon_;
+ // Ensure observed preferences are properly cleaned up.
+ PrefChangeRegistrar pref_registrar_;
+
DISALLOW_COPY_AND_ASSIGN(BackgroundModeManager);
};
« no previous file with comments | « no previous file | chrome/browser/background_mode_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698