Issue 2040 - Replaced localStorage with chrome.storage.local

chrome/ext/background.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 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 481 matching lines @@
   });
 
   // We have to ensure there is at least one listener for the onConnect event.
   // Otherwise we can't connect a port later, which we need to do in order to
   // detect when the extension is reloaded, disabled or uninstalled.
   chrome.runtime.onConnect.addListener(function() {});
 
 
   /* Storage */
 
-  ext.storage = localStorage;
+  ext.storage = {
+    get: function(keys, callback)
+    {
+      chrome.storage.local.get(keys, callback);
+    },
+    set: function(key, value, callback)
+    {
+      let items = {};
+      items[key] = value;
+      chrome.storage.local.set(items, callback);
+    },
+    remove: function(key, callback)
+    {
+      chrome.storage.local.remove(key, callback);
+    },
+    onChanged: chrome.storage.onChanged,
 
+    // Migrate localStorage to chrome.storage.local,
+    // ignoring unkown and inavlid preferences.

[kzar, 2015/04/13 10:26:39]

Nit: Typo in this comment "inavlid".

[Sebastian Noack, 2015/04/13 10:30:25]

Done.

+    migratePrefs: function(hooks)
+    {
+      var items = {};
+
+      for (let key in localStorage)
+      {
+        var item = hooks.map(key, localStorage[key]);
+        if (item)
+          items[item.key] = item.value;
+      }
+
+      chrome.storage.local.set(items, function() {
+        localStorage.clear();
+        hooks.done();
+      });
+    }
+  };
 
   /* Options */
 
   ext.showOptions = function(callback)
   {
     chrome.windows.getLastFocused(function(win)
     {
       var optionsUrl = chrome.extension.getURL("options.html");
       var queryInfo = {url: optionsUrl};
 
@@ skipping 16 matching lines @@
             callback(new Page(tab));
         }
         else
         {
           ext.pages.open(optionsUrl, callback);
         }
       });
     });
   };
 })();