Bug 1687235 - Enable ESLint rule no-setter-return for mobile/. r=agi

Depends on D102155

Differential Revision: https://phabricator.services.mozilla.com/D102156
This commit is contained in:
Mark Banner 2021-01-19 17:03:57 +00:00
parent aae922599a
commit 94aaf448b5
3 changed files with 2 additions and 13 deletions

View File

@ -660,14 +660,5 @@ module.exports = {
"mozilla/reject-chromeutils-import-null": "off",
},
},
{
files: [
"mobile/android/chrome/geckoview/SessionStateAggregator.js",
"mobile/android/modules/geckoview/DelayedInit.jsm",
],
rules: {
"no-setter-return": "off",
},
},
],
};

View File

@ -454,8 +454,6 @@ class MessageQueue extends Handler {
clearTimeout(this._timeout);
this._timeout = null;
}
return val;
}
uninit() {

View File

@ -177,12 +177,12 @@ var Impl = {
// we can get rid of our proxy property.
if (prop.get || prop.set) {
Object.defineProperty(object, name, prop);
return prop.set.call(object);
prop.set.call(object);
return;
}
prop.value = newVal;
Object.defineProperty(object, name, prop);
return newVal;
},
configurable: true,
enumerable: true,