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

Unified Diff: Source/modules/serviceworkers/CacheStorage.idl

Issue 424643002: Introducing the WebServiceWorkerCacheStorage (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: parameter naming 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: Source/modules/serviceworkers/CacheStorage.idl
diff --git a/Source/modules/serviceworkers/CacheStorage.idl b/Source/modules/serviceworkers/CacheStorage.idl
index 07b23386867fb95bf125e5f766997ae81a0c63d6..35b53933f927ae02493c6a18732f53acf5882091 100644
--- a/Source/modules/serviceworkers/CacheStorage.idl
+++ b/Source/modules/serviceworkers/CacheStorage.idl
@@ -8,11 +8,12 @@
Exposed=ServiceWorker,
RuntimeEnabled=ServiceWorker,
] interface CacheStorage {
- [CallWith=ScriptState, ImplementedAs=createFunction] Promise create(ScalarValueString key);
- [CallWith=ScriptState] Promise rename(ScalarValueString oldKey, ScalarValueString newKey);
- [CallWith=ScriptState] Promise get(ScalarValueString key);
- [CallWith=ScriptState, ImplementedAs=deleteFunction] Promise delete(ScalarValueString key);
+ [CallWith=ScriptState] Promise get(ScalarValueString cacheName);
+ [CallWith=ScriptState] Promise has(ScalarValueString cacheName);
+ [CallWith=ScriptState, ImplementedAs=createFunction] Promise create(ScalarValueString cacheName);
+ [CallWith=ScriptState, ImplementedAs=deleteFunction] Promise delete(ScalarValueString cacheName);
[CallWith=ScriptState] Promise keys();
- // FIXME: There are a number of other missing methods here and an active discussion ongoing on https://github.com/slightlyoff/ServiceWorker/issue/314
- // and https://github.com/slightlyoff/ServiceWorker/issues/315 about which should stay in. Once that discussion resolves, add the rest of the methods.
+ // FIXME: From https://github.com/slightlyoff/ServiceWorker/issues/372 and the updated
+ // https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#cache-storage ,
+ // we have yet to implement match().
};
« no previous file with comments | « Source/modules/serviceworkers/CacheStorage.cpp ('k') | Source/modules/serviceworkers/ServiceWorkerGlobalScope.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698