Chromium Code Reviews| Index: content/browser/cache_storage/cache_storage_manager.cc |
| diff --git a/content/browser/cache_storage/cache_storage_manager.cc b/content/browser/cache_storage/cache_storage_manager.cc |
| index 9b56c07954c8ffa895a80ad8e9bd548208eb2767..beb0080d001aa17587ac6393558c003f3e869c43 100644 |
| --- a/content/browser/cache_storage/cache_storage_manager.cc |
| +++ b/content/browser/cache_storage/cache_storage_manager.cc |
| @@ -173,11 +173,11 @@ void CacheStorageManager::MatchAllCaches( |
| } |
| void CacheStorageManager::SetBlobParametersForCache( |
| - net::URLRequestContext* request_context, |
| + const scoped_refptr<net::URLRequestContextGetter>& request_context, |
| base::WeakPtr<storage::BlobStorageContext> blob_storage_context) { |
| DCHECK_CURRENTLY_ON(BrowserThread::IO); |
| DCHECK(cache_storage_map_.empty()); |
| - DCHECK(!request_context_ || request_context_ == request_context); |
| + DCHECK(!request_context_ || request_context_.get() == request_context.get()); |
| DCHECK(!blob_context_ || blob_context_.get() == blob_storage_context.get()); |
| request_context_ = request_context; |
| blob_context_ = blob_storage_context; |
| @@ -292,7 +292,6 @@ CacheStorageManager::CacheStorageManager( |
| : root_path_(path), |
| cache_task_runner_(cache_task_runner), |
| quota_manager_proxy_(quota_manager_proxy), |
| - request_context_(NULL), |
| weak_ptr_factory_(this) { |
| if (quota_manager_proxy_.get()) { |
| quota_manager_proxy_->RegisterClient( |
| @@ -303,7 +302,7 @@ CacheStorageManager::CacheStorageManager( |
| CacheStorage* CacheStorageManager::FindOrCreateCacheStorage( |
| const GURL& origin) { |
| DCHECK_CURRENTLY_ON(BrowserThread::IO); |
| - DCHECK(request_context_); |
| + DCHECK(request_context_.get()); |
|
mmenke
2015/06/10 17:00:31
I don't think .get is needed to coerce refptrs to
jkarlin
2015/06/10 18:26:03
Done.
|
| CacheStorageMap::const_iterator it = cache_storage_map_.find(origin); |
| if (it == cache_storage_map_.end()) { |
| MigrateOrigin(origin); |