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

Unified Diff: safari/ext/background.js

Issue 29339112: Issue 3716 - Split up files stored in storage.local (Closed)
Patch Set: Fix the promise logic. Remove whitespace changes. Created April 1, 2016, 12:41 p.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
« lib/io.js ('K') | « lib/prefs.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
@@ -705,12 +705,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)
{
« lib/io.js ('K') | « lib/prefs.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld