diff js/modules/cache.js @ 81:e3de1fe32f40

made cache module require 'storage' instead of 'window'
author Atul Varma <avarma@mozilla.com>
date Sun, 25 Apr 2010 22:44:00 -0700
parents 4bb45ff5788a
children 00d23d6d41b4
line wrap: on
line diff
--- a/js/modules/cache.js	Sun Apr 25 22:26:14 2010 -0700
+++ b/js/modules/cache.js	Sun Apr 25 22:44:00 2010 -0700
@@ -1,8 +1,8 @@
 Require.modules["cache"] = function(exports, require) {
   const CACHE_NAME = "cache";
 
-  var window = require("window");
-  var cache = window.localStorage.getItem(CACHE_NAME);
+  var storage = require("storage");
+  var cache = storage.getItem(CACHE_NAME);
 
   if (cache)
     cache = JSON.parse(cache);
@@ -14,12 +14,12 @@
 
     // Remove the key first, to get around a strange iPad
     // issue: http://stackoverflow.com/questions/2603682/is-anyone-else-receiving-a-quota-exceeded-err-on-their-ipad-when-accessing-locals
-    window.localStorage.removeItem(CACHE_NAME);
+    storage.removeItem(CACHE_NAME);
 
     // TODO: We should really catch QUOTA_EXCEEDED_ERR here,
     // which could be thrown if the user is in private
     // browsing mode.
-    window.localStorage.setItem(CACHE_NAME, JSON.stringify(cache));
+    storage.setItem(CACHE_NAME, JSON.stringify(cache));
   };
 
   exports.get = function get(key) {
@@ -27,7 +27,7 @@
   };
 
   exports.clear = function clear() {
-    window.localStorage.removeItem(CACHE_NAME);
+    storage.removeItem(CACHE_NAME);
     cache = {};
   };
 };