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

Unified Diff: chrome/browser/extensions/extension_settings_backend.h

Issue 8375047: Separate the syncing of extension settings and app settings into separate data (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 9 years, 2 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/extensions/extension_settings_backend.h
diff --git a/chrome/browser/extensions/extension_settings_backend.h b/chrome/browser/extensions/extension_settings_backend.h
index 81c1067f580600f1e2875953228839efdee46306..3a0229d8c84a13229f4c6f8652c6b4f63850872d 100644
--- a/chrome/browser/extensions/extension_settings_backend.h
+++ b/chrome/browser/extensions/extension_settings_backend.h
@@ -26,8 +26,7 @@ class ExtensionSettingsBackend : public SyncableService {
// |observers| is the list of observers to settings changes.
explicit ExtensionSettingsBackend(
const FilePath& base_path,
- const scoped_refptr<ObserverListThreadSafe<ExtensionSettingsObserver> >&
- observers);
+ const scoped_refptr<ExtensionSettingsObserverList>& observers);
virtual ~ExtensionSettingsBackend();
@@ -41,7 +40,7 @@ class ExtensionSettingsBackend : public SyncableService {
const std::string& extension_id) const;
// Deletes all setting data for an extension. Call on the FILE thread.
- void DeleteExtensionData(const std::string& extension_id);
+ void DeleteStorage(const std::string& extension_id);
// Sends a change event to the observer list. |profile| is the profile which
// generated the change. Must be called on the FILE thread.
@@ -78,8 +77,7 @@ class ExtensionSettingsBackend : public SyncableService {
const FilePath base_path_;
// The list of observers to settings changes.
- const scoped_refptr<ObserverListThreadSafe<ExtensionSettingsObserver> >
- observers_;
+ const scoped_refptr<ExtensionSettingsObserverList> observers_;
// A cache of ExtensionSettingsStorage objects that have already been created.
// Ensure that there is only ever one created per extension.
@@ -87,6 +85,10 @@ class ExtensionSettingsBackend : public SyncableService {
StorageObjMap;
mutable StorageObjMap storage_objs_;
+ // Current sync model type. Will be UNSPECIFIED if sync hasn't been enabled
+ // yet, and either EXTENSION_SETTINGS or APP_SETTINGS if it has been.
+ syncable::ModelType sync_type_;
+
// Current sync processor, if any.
SyncChangeProcessor* sync_processor_;

Powered by Google App Engine
This is Rietveld 408576698