Split up extensions storage API implementations for sync and local storage.

chrome/browser/extensions/api/storage/local_value_store_cache.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.

[Devlin, 2014/03/05 23:30:53]

nit: I could be wrong, but I think the verdict for a file like this is that it's
considered "new", and should have an updated copyright. (Same for
sync_value_store_cache.)

[James Cook, 2014/03/05 23:45:19]

Done.

 // 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_LOCAL_VALUE_STORE_CACHE_H_
+#define CHROME_BROWSER_EXTENSIONS_API_STORAGE_LOCAL_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"
 
 namespace base {
 class FilePath;
 }
 
-namespace syncer {
-class SyncableService;
-}
-
 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 LOCAL namespace. It owns a backend for apps and
+// another for extensions. Each backend takes care of persistence.
+class LocalValueStoreCache : public ValueStoreCache {
  public:
-  SyncOrLocalValueStoreCache(
-      settings_namespace::Namespace settings_namespace,
-      const scoped_refptr<SettingsStorageFactory>& factory,
-      const SettingsStorageQuotaEnforcer::Limits& quota,
-      const scoped_refptr<SettingsObserverList>& observers,
-      const base::FilePath& profile_path);
-  virtual ~SyncOrLocalValueStoreCache();
-
-  syncer::SyncableService* GetSyncableService(syncer::ModelType type) const;
+  LocalValueStoreCache(const scoped_refptr<SettingsStorageFactory>& factory,
+                       const base::FilePath& profile_path);
+  virtual ~LocalValueStoreCache();
 
   // 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(LocalValueStoreCache);
 };
 
 }  // namespace extensions
 
-#endif  // CHROME_BROWSER_EXTENSIONS_API_STORAGE_SYNC_OR_LOCAL_VALUE_STORE_CACHE_H_
+#endif  // CHROME_BROWSER_EXTENSIONS_API_STORAGE_LOCAL_VALUE_STORE_CACHE_H_