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

chrome/browser/extensions/api/storage/settings_frontend.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/extensions/api/storage/settings_frontend.h"
 
-#include <limits>
-
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/files/file_path.h"
 #include "base/json/json_reader.h"
 #include "base/lazy_instance.h"
 #include "chrome/browser/extensions/api/storage/leveldb_settings_storage_factory.h"
-#include "chrome/browser/extensions/api/storage/sync_or_local_value_store_cache.h"
+#include "chrome/browser/extensions/api/storage/local_value_store_cache.h"
 #include "chrome/common/extensions/api/storage.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "extensions/browser/api/extensions_api_client.h"
 #include "extensions/browser/event_router.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/extension_system.h"
 
 using content::BrowserContext;
 using content::BrowserThread;
 
 namespace extensions {
 
-namespace storage = api::storage;
-
 namespace {
 
 base::LazyInstance<BrowserContextKeyedAPIFactory<SettingsFrontend> > g_factory =
     LAZY_INSTANCE_INITIALIZER;
 
 // Settings change Observer which forwards changes on to the extension
 // processes for |context| and its incognito partner if it exists.
 class DefaultObserver : public SettingsObserver {
  public:
   explicit DefaultObserver(BrowserContext* context)
       : browser_context_(context) {}
 
   // SettingsObserver implementation.
   virtual void OnSettingsChanged(
       const std::string& extension_id,
       settings_namespace::Namespace settings_namespace,
       const std::string& change_json) OVERRIDE {
     // TODO(gdk): This is a temporary hack while the refactoring for
     // string-based event payloads is removed. http://crbug.com/136045
     scoped_ptr<base::ListValue> args(new base::ListValue());
     args->Append(base::JSONReader::Read(change_json));
     args->Append(new base::StringValue(settings_namespace::ToString(
         settings_namespace)));
     scoped_ptr<Event> event(new Event(
-        storage::OnChanged::kEventName, args.Pass()));
+        api::storage::OnChanged::kEventName, args.Pass()));
     ExtensionSystem::Get(browser_context_)->event_router()->
         DispatchEventToExtension(extension_id, event.Pass());
   }
 
  private:
   BrowserContext* const browser_context_;
 };
 
-SettingsStorageQuotaEnforcer::Limits GetLocalLimits() {
-  SettingsStorageQuotaEnforcer::Limits limits = {
-    static_cast<size_t>(api::storage::local::QUOTA_BYTES),
-    std::numeric_limits<size_t>::max(),
-    std::numeric_limits<size_t>::max()
-  };
-  return limits;
-}
-
-SettingsStorageQuotaEnforcer::Limits GetSyncLimits() {
-  SettingsStorageQuotaEnforcer::Limits limits = {
-    static_cast<size_t>(api::storage::sync::QUOTA_BYTES),
-    static_cast<size_t>(api::storage::sync::QUOTA_BYTES_PER_ITEM),
-    static_cast<size_t>(api::storage::sync::MAX_ITEMS)
-  };
-  return limits;
-}
-
 }  // namespace
 
 // static
 SettingsFrontend* SettingsFrontend::Get(BrowserContext* context) {
   return BrowserContextKeyedAPIFactory<SettingsFrontend>::Get(context);
 }
 
 // static
 SettingsFrontend* SettingsFrontend::CreateForTesting(
     const scoped_refptr<SettingsStorageFactory>& storage_factory,
     BrowserContext* context) {
   return new SettingsFrontend(storage_factory, context);
 }
 
 SettingsFrontend::SettingsFrontend(BrowserContext* context)
-    : local_quota_limit_(GetLocalLimits()),
-      sync_quota_limit_(GetSyncLimits()),
-      browser_context_(context) {
+    : browser_context_(context) {
   Init(new LeveldbSettingsStorageFactory());
 }
 
 SettingsFrontend::SettingsFrontend(
     const scoped_refptr<SettingsStorageFactory>& factory,
     BrowserContext* context)
-    : local_quota_limit_(GetLocalLimits()),
-      sync_quota_limit_(GetSyncLimits()),
-      browser_context_(context) {
+    : browser_context_(context) {
   Init(factory);
 }
 
 void SettingsFrontend::Init(
     const scoped_refptr<SettingsStorageFactory>& factory) {
   observers_ = new SettingsObserverList();
   browser_context_observer_.reset(new DefaultObserver(browser_context_));
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!browser_context_->IsOffTheRecord());
 
   observers_->AddObserver(browser_context_observer_.get());
 
-  const base::FilePath& browser_context_path = browser_context_->GetPath();
   caches_[settings_namespace::LOCAL] =
-      new SyncOrLocalValueStoreCache(
-          settings_namespace::LOCAL,
-          factory,
-          local_quota_limit_,
-          observers_,
-          browser_context_path);
-  caches_[settings_namespace::SYNC] =
-      new SyncOrLocalValueStoreCache(
-          settings_namespace::SYNC,
-          factory,
-          sync_quota_limit_,
-          observers_,
-          browser_context_path);
+      new LocalValueStoreCache(factory, browser_context_->GetPath());

[Yoyo Zhou, 2014/03/06 00:43:22]

Why not do this one in the APIClient too?

[James Cook, 2014/03/06 00:47:14]

Do you mean to have the APIClient always be responsible for adding all the value
store caches, have ChromeExtensionsAPIClient add local/managed/sync and
introduce a ShellExtensionsAPIClient to add local?  Right now app_shell just
uses the stub implementation for ExtensionsAPIClient, but I could change that if
you think it would be better.

I'll still end up moving the LocalValueStoreCache implementation into
//extensions, since app_shell needs it regardless.

[Yoyo Zhou, 2014/03/06 01:22:54]

I don't have a strong preference. If it requires adding a
ShellExtensionsAPIClient then I say don't do it yet.

 
-  // Add any additional caches the embedder supports (for example, a cache
-  // for chrome.storage.managed).
+  // Add any additional caches the embedder supports (for example, caches
+  // for chrome.storage.managed and chrome.storage.sync).
   ExtensionsAPIClient::Get()->AddAdditionalValueStoreCaches(
       browser_context_, factory, observers_, &caches_);
 }
 
 SettingsFrontend::~SettingsFrontend() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   observers_->RemoveObserver(browser_context_observer_.get());
   for (CacheMap::iterator it = caches_.begin(); it != caches_.end(); ++it) {
     ValueStoreCache* cache = it->second;
     cache->ShutdownOnUI();
     BrowserThread::DeleteSoon(BrowserThread::FILE, FROM_HERE, cache);
   }
 }
 
-syncer::SyncableService* SettingsFrontend::GetBackendForSync(
-    syncer::ModelType type) const {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
-  CacheMap::const_iterator it = caches_.find(settings_namespace::SYNC);
-  DCHECK(it != caches_.end());
-  const SyncOrLocalValueStoreCache* sync_cache =
-      static_cast<const SyncOrLocalValueStoreCache*>(it->second);
-  DCHECK(type == syncer::APP_SETTINGS || type == syncer::EXTENSION_SETTINGS);
-  return sync_cache->GetSyncableService(type);
+ValueStoreCache* SettingsFrontend::GetValueStoreCache(
+    settings_namespace::Namespace settings_namespace) const {
+  CacheMap::const_iterator it = caches_.find(settings_namespace);
+  if (it != caches_.end())
+    return it->second;
+  return NULL;
 }
 
 bool SettingsFrontend::IsStorageEnabled(
     settings_namespace::Namespace settings_namespace) const {
   return caches_.find(settings_namespace) != caches_.end();
 }
 
 void SettingsFrontend::RunWithStorage(
     scoped_refptr<const Extension> extension,
     settings_namespace::Namespace settings_namespace,
@@ skipping 46 matching lines @@
 SettingsFrontend::GetFactoryInstance() {
   return g_factory.Pointer();
 }
 
 // static
 const char* SettingsFrontend::service_name() {
   return "SettingsFrontend";
 }
 
 }  // namespace extensions