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

Unified Diff: content/browser/service_worker/cache_storage_context_impl.cc

Issue 1039763002: Cache Storage: Move files to content/*/cache_storage, rename classes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: GN fix Created 5 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: content/browser/service_worker/cache_storage_context_impl.cc
diff --git a/content/browser/service_worker/cache_storage_context_impl.cc b/content/browser/service_worker/cache_storage_context_impl.cc
deleted file mode 100644
index 358d745aa5ac5115cf458a86891eca0ed67d2ab1..0000000000000000000000000000000000000000
--- a/content/browser/service_worker/cache_storage_context_impl.cc
+++ /dev/null
@@ -1,105 +0,0 @@
-// Copyright 2015 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 "content/browser/service_worker/cache_storage_context_impl.h"
-
-#include "base/bind.h"
-#include "base/files/file_path.h"
-#include "base/threading/sequenced_worker_pool.h"
-#include "content/browser/fileapi/chrome_blob_storage_context.h"
-#include "content/browser/service_worker/service_worker_cache_storage_manager.h"
-#include "content/public/browser/browser_context.h"
-#include "content/public/browser/browser_thread.h"
-#include "net/url_request/url_request_context_getter.h"
-#include "storage/browser/blob/blob_storage_context.h"
-#include "storage/browser/quota/quota_manager_proxy.h"
-#include "storage/browser/quota/special_storage_policy.h"
-
-namespace content {
-
-CacheStorageContextImpl::CacheStorageContextImpl(
- BrowserContext* browser_context) {
- DCHECK_CURRENTLY_ON(BrowserThread::UI);
-}
-
-CacheStorageContextImpl::~CacheStorageContextImpl() {
-}
-
-void CacheStorageContextImpl::Init(
- const base::FilePath& user_data_directory,
- storage::QuotaManagerProxy* quota_manager_proxy,
- storage::SpecialStoragePolicy* special_storage_policy) {
- DCHECK_CURRENTLY_ON(BrowserThread::UI);
-
- is_incognito_ = user_data_directory.empty();
- base::SequencedWorkerPool* pool = BrowserThread::GetBlockingPool();
- scoped_refptr<base::SequencedTaskRunner> cache_task_runner =
- pool->GetSequencedTaskRunnerWithShutdownBehavior(
- BrowserThread::GetBlockingPool()->GetSequenceToken(),
- base::SequencedWorkerPool::SKIP_ON_SHUTDOWN);
-
- // This thread-hopping antipattern is needed here for some unit tests, where
- // browser threads are collapsed the quota manager is initialized before the
- // posted task can register the quota client.
- // TODO: Fix the tests to let the quota manager initialize normally.
- if (BrowserThread::CurrentlyOn(BrowserThread::IO)) {
- CreateCacheStorageManager(user_data_directory, cache_task_runner,
- quota_manager_proxy, special_storage_policy);
- return;
- }
-
- BrowserThread::PostTask(
- BrowserThread::IO, FROM_HERE,
- base::Bind(&CacheStorageContextImpl::CreateCacheStorageManager, this,
- user_data_directory, cache_task_runner,
- make_scoped_refptr(quota_manager_proxy),
- make_scoped_refptr(special_storage_policy)));
-}
-
-void CacheStorageContextImpl::Shutdown() {
- DCHECK_CURRENTLY_ON(BrowserThread::UI);
-
- BrowserThread::PostTask(
- BrowserThread::IO, FROM_HERE,
- base::Bind(&CacheStorageContextImpl::ShutdownOnIO, this));
-}
-
-ServiceWorkerCacheStorageManager* CacheStorageContextImpl::cache_manager()
- const {
- DCHECK_CURRENTLY_ON(BrowserThread::IO);
- return cache_manager_.get();
-}
-
-void CacheStorageContextImpl::SetBlobParametersForCache(
- net::URLRequestContextGetter* request_context,
- ChromeBlobStorageContext* blob_storage_context) {
- DCHECK_CURRENTLY_ON(BrowserThread::IO);
-
- if (cache_manager_ && request_context && blob_storage_context) {
- cache_manager_->SetBlobParametersForCache(
- request_context->GetURLRequestContext(),
- blob_storage_context->context()->AsWeakPtr());
- }
-}
-
-void CacheStorageContextImpl::CreateCacheStorageManager(
- const base::FilePath& user_data_directory,
- const scoped_refptr<base::SequencedTaskRunner>& cache_task_runner,
- storage::QuotaManagerProxy* quota_manager_proxy,
- storage::SpecialStoragePolicy* special_storage_policy) {
- DCHECK_CURRENTLY_ON(BrowserThread::IO);
-
- DCHECK(!cache_manager_);
- cache_manager_ = ServiceWorkerCacheStorageManager::Create(
- user_data_directory, cache_task_runner.get(),
- make_scoped_refptr(quota_manager_proxy));
-}
-
-void CacheStorageContextImpl::ShutdownOnIO() {
- DCHECK_CURRENTLY_ON(BrowserThread::IO);
-
- cache_manager_.reset();
-}
-
-} // namespace content

Powered by Google App Engine
This is Rietveld 408576698