Initial ServiceWorker Cache API polyfill.

Source/modules/serviceworkers/polyfills/cachePolyfill.js

+// Copyright 2014 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.
+
+// A simple, incomplete implementation of the Cache API, intended to facilitate
+// end to end serviceworker testing.
+
+// FIXME: Support AbstractResponse/OpaqueResponse correctly.
+// FIXME: Serialize the cache.
+// FIXME: Implement CacheStorage API.
+// FIXME: Bind all function references.
+(function (global) {

[falken, 2014/06/06 17:02:45]

I don't see it in the style guide, but "function(...)" looks much more used than
"function (...)" in the codebase.

[gavinp, 2014/06/06 21:39:54]

Done.

+    var _castToRequest = function (item) {
+        if (typeof item === 'string') {
+            item = new Request({
+                url: item,
+            });
+        }
+        return item;
+    };
+
+    var Cache = function() {
+        // An object containing a property for each HTTP fetch method. Those
+        // referenced objects contain a property for each URL, which is the
+        // Response.
+        this.entriesByMethod = {};
+    };
+
+    // FIXME: Should this be in the spec?
+    Cache.prototype.keys = function() {
+        var that = this;
+
+        return Promise.resolve(Array.prototype.concat.apply([], Object.keys(this.entriesByMethod).map(function (method) {
+            return Object.keys(that.entriesByMethod[method]).map(function (url) {
+                return new Request({method: method, url: url});
+            });
+        })));
+    };
+
+    // FIXME: Implement this.
+    // FIXME: Should spec rename each --> forEach?
+    Cache.prototype.forEach = Promise.reject.bind(Promise, 'Cache.prototype.forEach() not implemented.');
+    Cache.prototype.each = Promise.reject.bind(Promise, 'Cache.prototype.each() not implemented.');
+
+    Cache.prototype.put = function (request, response) {
+        request = _castToRequest(request);
+
+        var that = this;
+        return new Promise(function (resolve, reject) {
+            if (!that.entriesByMethod.hasOwnProperty(request.method)) {
+                that.entriesByMethod[request.method] = {};
+            }
+
+            var entriesByUrl = that.entriesByMethod[request.method];
+            entriesByUrl[request.url] = response;
+
+            resolve();
+        });
+    };
+
+    // FIXME: Implement this.
+    Cache.prototype.add = Promise.reject.bind(Promise, 'Cache.prototype.add() not implemented.');
+
+    // FIXME: Add QueryParams argument.
+    Cache.prototype.delete = function (request) {
+        request = _castToRequest(request);
+
+        var that = this;
+        return new Promise(function (resolve, reject) {
+            if (!that.entriesByMethod.hasOwnProperty(request.method)) {
+                reject('not found');
+                return;
+            }
+
+            var entriesByUrl = that.entriesByMethod[request.method];
+            delete entriesByUrl[request.url];

[falken, 2014/06/06 17:02:45]

Should we reject if the URL isn't present, like we do for method?

[gavinp, 2014/06/06 21:39:54]

jsbell suggested favouring idempotence, and so I went with that; we now just
resolve in all cases.

+            resolve();
+        });
+    };
+
+    // FIXME: Add QueryParams argument.
+    Cache.prototype.match = function (request) {
+        request = _castToRequest(request);
+
+        var that = this;
+        return new Promise(function (resolve, reject) {
+            if (!that.entriesByMethod.hasOwnProperty(request.method)) {
+                reject('not found');
+                return;
+            }
+
+            var entriesByUrl = that.entriesByMethod[request.method];
+
+            if (!entriesByUrl.hasOwnProperty(request.url)) {
+                reject('not found');
+                return;
+            }
+
+            var entry = entriesByUrl[request.url];
+            resolve(entry);
+        });
+    };
+
+    // FIXME: Implement this.
+    Cache.prototype.matchAll = Promise.reject.bind(Promise, 'Cache.prototype.matchAll not implemented.');
+
+    global.Cache = global.Cache || Cache;
+}(self));  // window or worker global scope.