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

Differential Revision: https://phabricator.services.mozilla.com/D102156
dothq · Jan 26, 2021 · e3ee71f · e3ee71f
1 parent 07d7dc3
commit e3ee71f
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 13 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -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",
-      },
-    },
   ],
 };
diff --git a/mobile/android/chrome/geckoview/SessionStateAggregator.js b/mobile/android/chrome/geckoview/SessionStateAggregator.js
@@ -454,8 +454,6 @@ class MessageQueue extends Handler {
       clearTimeout(this._timeout);
       this._timeout = null;
     }
-
-    return val;
   }
 
   uninit() {

diff --git a/mobile/android/modules/geckoview/DelayedInit.jsm b/mobile/android/modules/geckoview/DelayedInit.jsm
@@ -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,