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

Unified Diff: chrome/browser/prefs/profile_pref_store_manager.cc

Issue 220203011: Changes requested by gab for previously committed CLs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Don't commit if we didn't make any changes. Created 6 years, 9 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
Index: chrome/browser/prefs/profile_pref_store_manager.cc
diff --git a/chrome/browser/prefs/profile_pref_store_manager.cc b/chrome/browser/prefs/profile_pref_store_manager.cc
index 40e63fa35e999924af61c35144bb8b7c1e8820b8..f907a5cae6b4ab7a63d94f4ee51d56d0674e0d66 100644
--- a/chrome/browser/prefs/profile_pref_store_manager.cc
+++ b/chrome/browser/prefs/profile_pref_store_manager.cc
@@ -132,23 +132,10 @@ class DictionaryPrefStore : public PrefStore {
DISALLOW_COPY_AND_ASSIGN(DictionaryPrefStore);
};
-} // namespace
-
-// TODO(erikwright): Enable this on Chrome OS and Android once MACs are moved
-// out of Local State. This will resolve a race condition on Android and a
-// privacy issue on ChromeOS. http://crbug.com/349158
-const bool ProfilePrefStoreManager::kPlatformSupportsPreferenceTracking =
-#if defined(OS_ANDROID) || defined(OS_CHROMEOS)
- false;
-#else
- true;
-#endif
-
// Waits for a PrefStore to be initialized and then initializes the
// corresponding PrefHashStore.
// The observer deletes itself when its work is completed.
-class ProfilePrefStoreManager::InitializeHashStoreObserver
- : public PrefStore::Observer {
+class InitializeHashStoreObserver : public PrefStore::Observer {
public:
// Creates an observer that will initialize |pref_hash_store| with the
// contents of |pref_store| when the latter is fully loaded.
@@ -179,15 +166,11 @@ class ProfilePrefStoreManager::InitializeHashStoreObserver
DISALLOW_COPY_AND_ASSIGN(InitializeHashStoreObserver);
};
-ProfilePrefStoreManager::InitializeHashStoreObserver::
- ~InitializeHashStoreObserver() {}
+InitializeHashStoreObserver::~InitializeHashStoreObserver() {}
-void ProfilePrefStoreManager::InitializeHashStoreObserver::OnPrefValueChanged(
- const std::string& key) {}
+void InitializeHashStoreObserver::OnPrefValueChanged(const std::string& key) {}
-void
-ProfilePrefStoreManager::InitializeHashStoreObserver::OnInitializationCompleted(
- bool succeeded) {
+void InitializeHashStoreObserver::OnInitializationCompleted(bool succeeded) {
// If we successfully loaded the preferences _and_ the PrefHashStoreImpl
// hasn't been initialized by someone else in the meantime, initialize it now.
const PrefHashStoreImpl::StoreVersion pre_update_version =
@@ -205,6 +188,18 @@ ProfilePrefStoreManager::InitializeHashStoreObserver::OnInitializationCompleted(
delete this;
}
+} // namespace
+
+// TODO(erikwright): Enable this on Chrome OS and Android once MACs are moved
+// out of Local State. This will resolve a race condition on Android and a
+// privacy issue on ChromeOS. http://crbug.com/349158
+const bool ProfilePrefStoreManager::kPlatformSupportsPreferenceTracking =
+#if defined(OS_ANDROID) || defined(OS_CHROMEOS)
+ false;
+#else
+ true;
+#endif
+
ProfilePrefStoreManager::ProfilePrefStoreManager(
const base::FilePath& profile_path,
const std::vector<PrefHashFilter::TrackedPreferenceMetadata>&
@@ -253,7 +248,7 @@ base::Time ProfilePrefStoreManager::GetResetTime(PrefService* pref_service) {
// PrefHashFilter::GetResetTime will read the value through the pref service,
// and thus through the SegregatedPrefStore. Even though it's not listed as
- // "protected" it will be read from the protected store prefentially to the
+ // "protected" it will be read from the protected store preferentially to the
// (NULL) value in the unprotected pref store.
return PrefHashFilter::GetResetTime(pref_service);
}
« no previous file with comments | « chrome/browser/prefs/profile_pref_store_manager.h ('k') | chrome/browser/prefs/profile_pref_store_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698