Bug 1687235 - Enable ESLint rule no-setter-return for devtools. r=jdescottes

Differential Revision: https://phabricator.services.mozilla.com/D102154
This commit is contained in:
Mark Banner 2021-01-26 13:44:01 +00:00
parent 930543e681
commit 6f70e91a5d
4 changed files with 4 additions and 6 deletions

View File

@ -662,9 +662,6 @@ module.exports = {
},
{
files: [
"devtools/client/shared/async-store-helper.js",
"devtools/client/shared/prefs.js",
"devtools/shared/task.js",
"dom/base/SlowScriptDebug.jsm",
"dom/media/PeerConnection.jsm",
"mobile/android/chrome/geckoview/SessionStateAggregator.js",

View File

@ -35,7 +35,7 @@ function asyncStoreHelper(root, mappings) {
return value || getMappingDefaultValue(key);
},
set(value) {
return asyncStorage.setItem(`${root}.${getMappingKey(key)}`, value);
asyncStorage.setItem(`${root}.${getMappingKey(key)}`, value);
},
})
);

View File

@ -144,7 +144,9 @@ function map(
Object.defineProperty(self, accessorName, {
get: () =>
serializer.in(get(cache, prefType, prefsRoot, prefName, fallbackValue)),
set: e => set(cache, prefType, prefsRoot, prefName, serializer.out(e)),
set: e => {
set(cache, prefType, prefsRoot, prefName, serializer.out(e));
},
});
}

View File

@ -480,7 +480,6 @@ Task.Debugging = {
gCurrentTask = null;
}
gMaintainStack = x;
return x;
},
/**