Merge pull request #2 from GoogleChrome/precache-promises

Allow items passed to preCache to be Promises for URLs as well as just URLs
wibblymat · May 5, 2015 · 85de7e3 · 85de7e3
2 parents 6752ca9 + 79def66
commit 85de7e3
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/lib/sw-toolbox.js b/lib/sw-toolbox.js
@@ -32,7 +32,7 @@ self.addEventListener('install', function(event) {
   helpers.debug('preCache list: ' + (options.preCacheItems.join(', ') || '(none)'));
   event.waitUntil(
     helpers.openCache({cacheName: inactiveCache}).then(function(cache) {
-      return cache.addAll(options.preCacheItems);
+      return Promise.all(options.preCacheItems).then(cache.addAll.bind(cache));
     })
   );
 });

diff --git a/sw-toolbox.js b/sw-toolbox.js