Issue 4023 - Move storage of subscription lists to localStorage

lib/io.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
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+"use strict";
+
 const keyPrefix = "file:";
 
 function fileToKey(file)
 {
   return keyPrefix + (file instanceof FakeFile ? file.path : file.spec);
 }
 
-function loadFile(file, successCallback, errorCallback)
+function loadFile(file)
 {
-  let key = fileToKey(file);
-
-  ext.storage.get([key], function(items)
+  return new Promise((resolve, reject) =>
   {
-    let entry = items[key];
-
-    if (entry)
-      successCallback(entry);
-    else
-      errorCallback(new Error("File doesn't exist"));
+    let key = fileToKey(file);
+    ext.storage.get([key], items =>
+    {
+      let entry = items[key];
+      if (!entry)
+      {
+        // Check in localStorage
+        try
+        {
+          entry = JSON.parse(window.localStorage.getItem(key));
+        }
+        catch(err) {}
+      }
+      if (entry)
+        resolve(entry);
+      else
+        reject(new Error("File doesn't exist"));
+    });
   });
 }
 
-function saveFile(file, data, callback)
-{
-  ext.storage.set(
-    fileToKey(file),
-    {
-      content: data,
-      lastModified: Date.now()
-    },
-    callback
-  );
-}
-
 exports.IO =
 {
   resolveFilePath: function(path)
   {
     return new FakeFile(path);
   },
 
   readFromFile: function(file, listener, callback)
   {
     function onLoaded(entry)
     {
-      for (let line of entry.content)
-        listener.process(line);
-
-      listener.process(null);
+      if ("content" in entry)
+      {
+        if ("compressed" in entry)
+          entry.content = JSON.parse(LZString.decompressFromUTF16(entry.content));
+        for (let line of entry.content)
+          listener.process(line);
+      }
       callback(null);
     }
 
-    loadFile(file, onLoaded, callback);
+    loadFile(file).then(onLoaded, callback);
   },
 
   writeToFile: function(file, data, callback)
   {
-    saveFile(file, data, callback);
-  },
+    let entry = {};
+    let key = fileToKey(file);
 
-  copyFile: function(fromFile, toFile, callback)
-  {
-    function onLoaded(entry)
+    // Edge cannot write files larger than 1Mb (500k string in UTF-16).

[kzar, 2016/08/30 14:13:02]

I think this comment should be inside the else block, since that's the Edge
related code.

[Oleksandr, 2016/08/31 21:54:37]

Done.

+    // However it does not always raise an exception when trying to do so.
+    if (typeof browser == "undefined")
     {
-      saveFile(toFile, entry.content, callback);
+      entry[key] = {lastModified: Date.now(), 

[kzar, 2016/08/30 14:13:03]

Nit: No need to wrap this into two lines if it will fit on one.

[Oleksandr, 2016/08/31 21:54:37]

Done.

+                    content: data};
+      ext.storage.set(entry, callback);
     }
-
-    loadFile(fromFile, onLoaded, callback);
-  },
-
-  renameFile: function(fromFile, newName, callback)
-  {
-    function onLoaded()
+    else
     {
-      ext.storage.remove(fileToKey(fromFile), function()
-      {
-        ext.storage.set(keyPrefix + newName, entry, callback);
-      });
+      // Compress and fallback to localStorage
+      let processedData = LZString.compressToUTF16(JSON.stringify(data));
+      ext.storage.remove(key);
+      entry[key] = {lastModified: Date.now(), 

[kzar, 2016/08/30 14:13:02]

Nit: This could fit on two lines instead of three.

[Oleksandr, 2016/08/31 21:54:37]

Done. However it looked better as it was, IMHO.

[kzar, 2016/09/01 12:57:37]

I can't see the change?

+                    content: processedData,
+                    compressed: true};
+      window.localStorage.setItem(key, JSON.stringify(entry[key]));
+      callback();
     }
-
-    loadFile(fromFile, 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, onLoaded, callback);
+    loadFile(file).then(onLoaded, callback);
   }
 };