Issue 4023 - Use localforage (IndexedDB) instead of localStorage

lib/adblockplus.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 339 matching lines @@
 require.scopes["io"] = (function()
 {
   var exports = {};
   var keyPrefix = "file:";
 
   function fileToKey(file)
   {
     return keyPrefix + (file instanceof FakeFile ? file.path : file.spec);
   }
 
-  function loadFile(file)
-  {
-    return new Promise(function(resolve, reject)
-    {
-      var key = fileToKey(file);
-      ext.storage.get([key], function(items)
-      {
-        var entry = items[key];
-        if (entry)
-        {
-          resolve(entry);
-        }
-        else
-        {
-          try
-          {
-            entry = localforage.getItem(key, function(err, value) 
-            {
-              if (err || !value)
-                reject(new Error("File doesn't exist"));                
-              else
-                resolve(value);
-            });
-          }
-          catch (err)
-          {}
-        }
-      });
-    }.bind(this));
-  }
+  function loadFile(file, successCallback, errorCallback)
+  {
+    var key = fileToKey(file);
+    // Make sure we do not have subscriptions in localStorage from older versions first
+    var entry = localStorage.getItem(key);
+    if (typeof entry == "string")
+    {
+      try
+      {
+        entry = JSON.parse(entry);
+      }
+      catch(err) 
+      {
+        setTimeout(errorCallback(new Error("File is corrupted")));
+        return;
+      }
+      setTimeout(successCallback(entry));
+      return;
+    }
+    // Now try to read from IndexedDB
+    localforage.getItem(key, function(err, value) 
+    {
+      if (err || !value)
+        errorCallback(new Error("File doesn't exist"));                
+      else
+        successCallback(value);
+    });
+  }
+
   function saveFile(file, data, callback)
   {
+    var key = fileToKey(file);
     var entry = {
       lastModified: Date.now(),
       content: data
     };
 
-    ext.storage.remove(fileToKey(file));
+    localStorage.removeItem(key);
     localforage.setItem(key, entry, callback);

[kzar, 2016/10/05 13:46:53]

Did you test this? Isn't `key` undefined now?

[kzar, 2016/10/05 13:46:53]

Why not just pass in `{lastModified: Date.now(), content: data}` here directly
if you're not using the `entry` variable elsewhere any more?

[Oleksandr, 2016/10/05 21:07:40]

I am not sure this is prettier, but I don't have a strong opinion.

[Sebastian Noack, 2016/10/05 21:27:31]

Also considering that we'd have to wrap this line if we inline the object
literal, I think it is fine as it is.

[Sebastian Noack, 2016/10/05 21:28:38]

...as it was, I meant.

[Oleksandr, 2016/10/05 21:49:12]

Done.

   }
   exports.IO = {
     resolveFilePath: function(path)
     {
       return new FakeFile(path);
     },
     readFromFile: function(file, listener, callback)
     {
       function onLoaded(entry)
       {
         if ("content" in entry)
         {
           for (var _loopIndex15 = 0; _loopIndex15 < entry.content.length; ++_loopIndex15)
           {
             var line = entry.content[_loopIndex15];
             listener.process(line);
           }
         }
         callback(null);
       }
-      loadFile(file).then(onLoaded, callback);
+      loadFile(file, onLoaded, callback);
     },
     writeToFile: function(file, data, callback)
     {
       saveFile(file, data, callback);
     },
     copyFile: function(fromFile, toFile, callback)
     {
       function onLoaded(entry)
       {
         saveFile(toFile, entry.content, callback);
       }
-      loadFile(file).then(onLoaded, callback);
+      loadFile(file, onLoaded, callback);
     },
     renameFile: function(fromFile, newName, callback)
     {
       function onLoaded()
       {
         ext.storage.remove(fileToKey(fromFile), function()
         {
           ext.storage.set(keyPrefix + newName, entry, callback);
         });
       }
-      loadFile(file).then(onLoaded, callback);
+      loadFile(file, onLoaded, callback);
     },
     removeFile: function(file, callback)
     {
       ext.storage.remove(fileToKey(file), callback);
     },
     statFile: function(file, callback)
     {
       function onLoaded(entry)
       {
         callback(null,
         {
           exists: true,
           lastModified: entry.lastModified
         });
       }
-      loadFile(file).then(onLoaded, callback);
+      loadFile(file, onLoaded, callback);
     }
   };
   return exports;
 })();
 require.scopes["downloader"] = (function()
 {
   var exports = {};
   var Utils = require("utils").Utils;
   var MILLIS_IN_SECOND = exports.MILLIS_IN_SECOND = 1000;
   var MILLIS_IN_MINUTE = exports.MILLIS_IN_MINUTE = 60 * MILLIS_IN_SECOND;
@@ skipping 6088 matching lines @@
     search.push("notificationDownloadCount=" + encodeURIComponent(downlCount));
     chrome.runtime.setUninstallURL(Utils.getDocLink("uninstalled") + "&" + search.join("&"));
   }
   if ("setUninstallURL" in chrome.runtime)
   {
     Prefs.untilLoaded.then(setUninstallURL);
     Prefs.on("notificationdata", setUninstallURL);
   }
   return exports;
 })();