Bug 1320186: Don't sanitize storage values more than once. r?aswan
MozReview-Commit-ID: y6S5LqS7fL
--- a/toolkit/components/extensions/ExtensionStorage.jsm
+++ b/toolkit/components/extensions/ExtensionStorage.jsm
@@ -137,21 +137,21 @@ this.ExtensionStorage = {
"ExtensionStorage: Finish writing extension data",
promise);
return promise.then(() => {
AsyncShutdown.profileBeforeChange.removeBlocker(promise);
});
},
- set(extensionId, items, context) {
+ set(extensionId, items) {
return this.read(extensionId).then(extData => {
let changes = {};
for (let prop in items) {
- let item = this.sanitize(items[prop], context);
+ let item = items[prop];
changes[prop] = {oldValue: extData[prop], newValue: item};
extData[prop] = item;
}
this.notifyListeners(extensionId, changes);
return this.write(extensionId);
});
--- a/toolkit/components/extensions/ext-storage.js
+++ b/toolkit/components/extensions/ext-storage.js
@@ -26,17 +26,17 @@ this.storage = class extends ExtensionAP
let {extension} = context;
return {
storage: {
local: {
get: function(spec) {
return ExtensionStorage.get(extension.id, spec);
},
set: function(items) {
- return ExtensionStorage.set(extension.id, items, context);
+ return ExtensionStorage.set(extension.id, items);
},
remove: function(keys) {
return ExtensionStorage.remove(extension.id, keys);
},
clear: function() {
return ExtensionStorage.clear(extension.id);
},
},