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

Unified Diff: content/browser/renderer_host/render_message_filter.cc

Issue 2056983004: [CacheStorage] Give ownership of all CacheStorageCaches to CacheStorage (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 6 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/renderer_host/render_message_filter.cc
diff --git a/content/browser/renderer_host/render_message_filter.cc b/content/browser/renderer_host/render_message_filter.cc
index aab4ef993663dd98e107a57e2d7a7f32593e17f0..8529988bf918fc04e904f97d34233c00e9a34e04 100644
--- a/content/browser/renderer_host/render_message_filter.cc
+++ b/content/browser/renderer_host/render_message_filter.cc
@@ -23,6 +23,7 @@
#include "content/browser/blob_storage/chrome_blob_storage_context.h"
#include "content/browser/browser_main_loop.h"
#include "content/browser/cache_storage/cache_storage_cache.h"
+#include "content/browser/cache_storage/cache_storage_cache_handle.h"
#include "content/browser/cache_storage/cache_storage_context_impl.h"
#include "content/browser/cache_storage/cache_storage_manager.h"
#include "content/browser/dom_storage/dom_storage_context_wrapper.h"
@@ -124,7 +125,9 @@ void ResizeHelperPostMsgToUIThread(int render_process_id,
}
#endif
-void NoOpCacheStorageErrorCallback(CacheStorageError error) {}
+void NoOpCacheStorageErrorCallback(
+ std::unique_ptr<CacheStorageCacheHandle> cache_handle,
+ CacheStorageError error) {}
} // namespace
@@ -536,12 +539,16 @@ void RenderMessageFilter::OnCacheStorageOpenCallback(
base::Time expected_response_time,
scoped_refptr<net::IOBuffer> buf,
int buf_len,
- scoped_refptr<CacheStorageCache> cache,
+ std::unique_ptr<CacheStorageCacheHandle> cache_handle,
CacheStorageError error) {
- if (error != CACHE_STORAGE_OK)
+ if (error != CACHE_STORAGE_OK || !cache_handle || !cache_handle->value())
return;
- cache->WriteSideData(base::Bind(&NoOpCacheStorageErrorCallback), url,
- expected_response_time, buf, buf_len);
+ CacheStorageCache* cache = cache_handle->value();
+ if (!cache)
+ return;
+ cache->WriteSideData(base::Bind(&NoOpCacheStorageErrorCallback,
+ base::Passed(std::move(cache_handle))),
+ url, expected_response_time, buf, buf_len);
}
void RenderMessageFilter::OnKeygen(uint32_t key_size_index,
« no previous file with comments | « content/browser/renderer_host/render_message_filter.h ('k') | content/browser/service_worker/service_worker_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698