Issue 2040 - Replaced localStorage with chrome.storage.local

safari/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 629 matching lines @@
         // only when reloading the page or following links, but not when
         // the current page is replaced with a prerendered page.
         replacePage(pages[event.message.pageId]);
         break;
     }
   });
 
 
   /* Storage */
 
-  ext.storage = safari.extension.settings;
+  ext.storage = {
+    get: function(keys, callback)
+    {
+      var items = {};
+      var settings = safari.extension.settings;
+
+      for (var i = 0; i < keys.length; i++)
+      {
+        var key = keys[i];
+        if (key in settings)
+          items[key] = settings[key];
+      }
+
+      setTimeout(callback, 0, items);

[Wladimir Palant, 2015/03/12 23:05:37]

Did you verify that this works on all Safari versions we support? MDN doesn't
know when Safari added support for additional setTimeout parameters.

[Sebastian Noack, 2015/03/12 23:35:29]

I saw this one coming. I checked MDN and made the same observation as you did.
So I tried on Safari 5.1 (the oldest version ABP runs on) and Safari 8 (the
newest one). And also to my surprise that is actually supported. :D

+    },
+    set: function(key, value, callback)
+    {
+      safari.extension.settings[key] = value;
+
+      if (callback)
+        setTimeout(callback, 0);
+    },
+    remove: function(key, callback)
+    {
+      delete safari.extension.settings[key];
+
+      if (callback)
+        setTimeout(callback, 0);
+    },
+    onChanged: new ext._EventTarget(),
+
+    // Preferences were previously encoded as JSON for compatibility
+    // with localStorage, which has been used on Chrome.
+    migratePrefs: function(prefs)
+    {
+      var settings = safari.extension.settings;
+
+      for (var key in settings)
+      {
+        if (key in prefs)
+        {
+          try
+          {
+            settings["pref:" + key] = JSON.parse(settings[key]);
+          }
+          catch (e)
+          {
+          }
+
+          delete settings[key];
+        }
+      }
+    }
+  };
+
+  safari.extension.settings.addEventListener("change", function(event)
+  {
+    var changes = {};
+    var change = changes[event.key] = {};
+
+    if (event.oldValue != null)
+      change.oldValue = event.oldValue;
+    if (event.newValue != null)
+      change.newValue = event.newValue;
+
+    ext.storage.onChanged._dispatch(changes);
+  });
 
 
   /* Options */
 
   ext.showOptions = function(callback)
   {
     var optionsUrl = safari.extension.baseURI + "options.html";
 
     for (var id in pages)
     {
       var page = pages[id];
       var tab = page._tab;
 
       if (page.url.href == optionsUrl && tab.browserWindow == safari.application.activeBrowserWindow)
       {
         tab.activate();
         if (callback)
           callback(page);
         return;
       }
     }
 
     ext.pages.open(optionsUrl, callback);
   };
 })();