Mercurial > bugzilla-dashboard
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 = {}; }; };