Signed settings refactoring: Proper caching and more tests.

chrome/browser/chromeos/cros_settings.h

Index: chrome/browser/chromeos/cros_settings.h
diff --git a/chrome/browser/chromeos/cros_settings.h b/chrome/browser/chromeos/cros_settings.h
index c7f66d276ce0fcedd479f57912bee74fd6fb2d7e..c5602635f1eae6b7fabcd44379bf7007687ccded 100644
--- a/chrome/browser/chromeos/cros_settings.h
+++ b/chrome/browser/chromeos/cros_settings.h
@@ -92,6 +92,10 @@ class CrosSettings : public base::NonThreadSafe {
   // Returns the provider that handles settings with the path or prefix.
   CrosSettingsProvider* GetProvider(const std::string& path) const;
 
+  // Forces all providers to reload their caches from the respective backing
+  // stores if they have any.
+  void ReloadProviders() const;

[Denis Lagno, 2011/12/01 00:38:12]

just wondering why is should be const?

[pastarmovj, 2011/12/01 10:07:17]

No special reason but it doesn't change the internal state of this class so I
decided to mark it as such.

+
  private:
   // List of ChromeOS system settings providers.
   std::vector<CrosSettingsProvider*> providers_;