Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: safari/ext/background.js

Issue 29350042: Issue 4023 - Move storage of subscription lists to localStorage (Closed)
Patch Set: Cleanup Created Aug. 25, 2016, 1:57 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« qunit/tests/io.js ('K') | « qunit/tests/io.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments 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)
{
- 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 »

Powered by Google App Engine
This is Rietveld