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

Unified Diff: chrome/browser/extensions/api/storage/sync_value_store_cache.h

Issue 184043031: Split up extensions storage API implementations for sync and local storage. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase (split_sync) 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/extensions/api/storage/sync_value_store_cache.h
diff --git a/chrome/browser/extensions/api/storage/sync_or_local_value_store_cache.h b/chrome/browser/extensions/api/storage/sync_value_store_cache.h
similarity index 59%
rename from chrome/browser/extensions/api/storage/sync_or_local_value_store_cache.h
rename to chrome/browser/extensions/api/storage/sync_value_store_cache.h
index 4cd9e37d8b183172dded552f7e4cc8d4f58dbe18..12a97251bb59529ca7ccf83b688898844d09c6f0 100644
--- a/chrome/browser/extensions/api/storage/sync_or_local_value_store_cache.h
+++ b/chrome/browser/extensions/api/storage/sync_value_store_cache.h
@@ -1,16 +1,15 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright 2014 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNC_OR_LOCAL_VALUE_STORE_CACHE_H_
-#define CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNC_OR_LOCAL_VALUE_STORE_CACHE_H_
+#ifndef CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNC_VALUE_STORE_CACHE_H_
+#define CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNC_VALUE_STORE_CACHE_H_
#include "base/basictypes.h"
#include "base/compiler_specific.h"
#include "base/memory/ref_counted.h"
#include "base/memory/scoped_ptr.h"
#include "chrome/browser/extensions/api/storage/settings_observer.h"
-#include "chrome/browser/extensions/api/storage/settings_storage_quota_enforcer.h"
#include "chrome/browser/extensions/api/storage/value_store_cache.h"
#include "sync/api/syncable_service.h"
@@ -27,43 +26,36 @@ namespace extensions {
class SettingsBackend;
class SettingsStorageFactory;
-// ValueStoreCache for the LOCAL and SYNC namespaces. It owns a backend for
-// apps and another for extensions. Each backend takes care of persistence and
-// syncing.
-class SyncOrLocalValueStoreCache : public ValueStoreCache {
+// ValueStoreCache for the SYNC namespace. It owns a backend for apps and
+// another for extensions. Each backend takes care of persistence and syncing.
+class SyncValueStoreCache : public ValueStoreCache {
public:
- SyncOrLocalValueStoreCache(
- settings_namespace::Namespace settings_namespace,
+ SyncValueStoreCache(
const scoped_refptr<SettingsStorageFactory>& factory,
- const SettingsStorageQuotaEnforcer::Limits& quota,
const scoped_refptr<SettingsObserverList>& observers,
const base::FilePath& profile_path);
- virtual ~SyncOrLocalValueStoreCache();
+ virtual ~SyncValueStoreCache();
syncer::SyncableService* GetSyncableService(syncer::ModelType type) const;
// ValueStoreCache implementation:
-
virtual void RunWithValueStoreForExtension(
const StorageCallback& callback,
scoped_refptr<const Extension> extension) OVERRIDE;
-
virtual void DeleteStorageSoon(const std::string& extension_id) OVERRIDE;
private:
void InitOnFileThread(const scoped_refptr<SettingsStorageFactory>& factory,
- const SettingsStorageQuotaEnforcer::Limits& quota,
const scoped_refptr<SettingsObserverList>& observers,
const base::FilePath& profile_path);
- settings_namespace::Namespace settings_namespace_;
bool initialized_;
scoped_ptr<SettingsBackend> app_backend_;
scoped_ptr<SettingsBackend> extension_backend_;
- DISALLOW_COPY_AND_ASSIGN(SyncOrLocalValueStoreCache);
+ DISALLOW_COPY_AND_ASSIGN(SyncValueStoreCache);
};
} // namespace extensions
-#endif // CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNC_OR_LOCAL_VALUE_STORE_CACHE_H_
+#endif // CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNC_VALUE_STORE_CACHE_H_

Powered by Google App Engine
This is Rietveld 408576698