PreferencesManager

ash/common/wm_shell.h

Index: ash/common/wm_shell.h
diff --git a/ash/common/wm_shell.h b/ash/common/wm_shell.h
index 2c7a1548972c36a3c193ec513650f6f6a71cd521..f2d290f27b32de29597895ccf16aa09a415e6e8a 100644
--- a/ash/common/wm_shell.h
+++ b/ash/common/wm_shell.h
@@ -21,6 +21,8 @@
 #include "ui/compositor/layer_type.h"
 #include "ui/wm/public/window_types.h"
 
+class PrefObserverStore;
+
 namespace base {
 class SequencedWorkerPool;
 }
@@ -152,6 +154,8 @@ class ASH_EXPORT WmShell {
 
   PaletteDelegate* palette_delegate() { return palette_delegate_.get(); }
 
+  PrefObserverStore* pref_store() { return pref_store_.get(); }
+
   ShelfController* shelf_controller() { return shelf_controller_.get(); }
 
   ShelfDelegate* shelf_delegate() { return shelf_delegate_.get(); }
@@ -487,6 +491,7 @@ class ASH_EXPORT WmShell {
   std::unique_ptr<MruWindowTracker> mru_window_tracker_;
   std::unique_ptr<mojom::NewWindowClient> new_window_client_;
   std::unique_ptr<PaletteDelegate> palette_delegate_;
+  scoped_refptr<PrefObserverStore> pref_store_;

[James Cook, 2016/11/19 00:33:19]

Sorry, I meant, store this as a scoped_refptr<PrefStore>. But that may not work
if you need to call Init() in client code.

[jonross, 2016/11/19 00:35:36]

Ah, yeah.

I had planned for client code to call Init. That way each component can register
on demand.

The chrome side has a few monolithic spots where it registers all prefs. We
could do something similar, however I am hesitant as that has lead to issues
with bringing up unit tests.

   std::unique_ptr<ShelfController> shelf_controller_;
   std::unique_ptr<ShelfDelegate> shelf_delegate_;
   std::unique_ptr<ShelfWindowWatcher> shelf_window_watcher_;