Unified Diff: safari/ext/background.js
Issue 29350042:
Issue 4023 - Move storage of subscription lists to localStorage (Closed)
Patch Set:
Created Aug. 22, 2016, 9:24 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« qunit/tests/io.js ('K')
|
« qunit/tests/io.js ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: safari/ext/background.js |
| =================================================================== |
| --- a/safari/ext/background.js |
| +++ b/safari/ext/background.js |
| @@ -540,12 +540,12 @@ |
| setTimeout(callback, 0, items); |
| }, |
| - set: function(key, value, callback) |
| + set: function(items, callback) |
|
kzar
2016/08/22 15:38:33
I guess it's good not to break stuff, but if these
Oleksandr
2016/08/31 21:54:37
The idea is to commit with a bookmark to adblockpl
|
| { |
| - safari.extension.settings[key] = value; |
| - |
| + for (let key in items) |
| + safari.extension.settings[key] = items[key]; |
| if (callback) |
| - setTimeout(callback, 0); |
| + setTimeout(callback); |
| }, |
| remove: function(key, callback) |
| { |
« qunit/tests/io.js ('K')
|
« qunit/tests/io.js ('k')
|
no next file »
|
no next file with comments »
