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

Unified Diff: content/browser/service_worker/service_worker_fetch_stores_manager.h

Issue 424863002: Fill in some of ServiceWorkerFetchStores. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@cache_storage
Patch Set: Asynchronous cache init Created 6 years, 4 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/service_worker_fetch_stores_manager.h
diff --git a/content/browser/service_worker/service_worker_fetch_stores_manager.h b/content/browser/service_worker/service_worker_fetch_stores_manager.h
index 44c999aeba63ffa36bec95dc8e2dd56009a6d345..10139e1e179714f00614e65717f63f6cc02a9fef 100644
--- a/content/browser/service_worker/service_worker_fetch_stores_manager.h
+++ b/content/browser/service_worker/service_worker_fetch_stores_manager.h
@@ -22,6 +22,8 @@ namespace content {
// Keeps track of a ServiceWorkerFetchStores per origin. There is one
// ServiceWorkerFetchStoresManager per ServiceWorkerContextCore.
+// TODO(jkarlin): Remove ServiceWorkerFetchStores from memory once they're no
+// longer in active use.
michaeln 2014/08/07 00:10:13 Representing a store with a primitive 'int' at thi
jkarlin 2014/08/07 16:02:35 Acknowledged.
class CONTENT_EXPORT ServiceWorkerFetchStoresManager {
public:
static scoped_ptr<ServiceWorkerFetchStoresManager> Create(
@@ -47,7 +49,7 @@ class CONTENT_EXPORT ServiceWorkerFetchStoresManager {
const ServiceWorkerFetchStores::BoolAndErrorCallback& callback);
void Delete(const GURL& origin,
const std::string& key,
- const ServiceWorkerFetchStores::StoreAndErrorCallback& callback);
+ const ServiceWorkerFetchStores::BoolAndErrorCallback& callback);
void Keys(const GURL& origin,
const ServiceWorkerFetchStores::StringsAndErrorCallback& callback);
// TODO(jkarlin): Add match() function.

Powered by Google App Engine
This is Rietveld 408576698