Bug 1652690 - Remove outdated eslint rule for devtools/shared/platform r=nchevobbe,perftest-reviewers,sparky

Small cleanup of eslint rules.

Differential Revision: https://phabricator.services.mozilla.com/D83477

devtools/.eslintrc.js

@@ -147,13 +147,6 @@ module.exports = {
     "mozilla/no-aArgs": "error",
     // See bug 1224289.
     "mozilla/reject-importGlobalProperties": ["error", "everything"],
-    // devtools/shared/platform is special; see the README.md in that
-    // directory for details.  We reject requires using explicit
-    // subdirectories of this directory.
-    "mozilla/reject-some-requires": [
-      "error",
-      "^devtools/shared/platform/(chome|content)/",
-    ],
     "mozilla/var-only-at-top-level": "error",
     "mozilla/use-chromeutils-import": ["error", { allowCu: true }],
 

devtools/client/inspector/.eslintrc.js

@@ -9,13 +9,13 @@ module.exports = {
   extends: "../../.eslintrc.js",
 
   rules: {
-    // The inspector is being migrated to HTML and cleaned of
-    // chrome-privileged code, so this rule disallows requiring chrome
-    // code. Some files in the inspector disable this rule still. The
-    // goal is to enable the rule globally on all files.
+    // This rule was introduced for the DevTools HTML initiative, the goal was
+    // to avoid requiring helpers unavailable in a regular content page, to
+    // use DevTools as a regular webapplication. Should be reviewed in
+    // Bug 1591091.
     "mozilla/reject-some-requires": [
       "error",
-      "^(chrome|chrome:.*|resource:.*|devtools/server/.*|.*\\.jsm|devtools/shared/platform/(chome|content)/.*)$",
+      "^(chrome|chrome:.*|resource:.*|devtools/server/.*|.*\\.jsm)$",
     ],
   },
 };

devtools/client/netmonitor/.eslintrc.js

@@ -6,13 +6,13 @@
 
 module.exports = {
   rules: {
-    // The netmonitor is being migrated to HTML and cleaned of
-    // chrome-privileged code, so this rule disallows requiring chrome
-    // code. Some files in the netmonitor disable this rule still. The
-    // goal is to enable the rule globally on all files.
+    // This rule was introduced for the DevTools HTML initiative, the goal was
+    // to avoid requiring helpers unavailable in a regular content page, to
+    // use DevTools as a regular webapplication. Should be reviewed in
+    // Bug 1591091.
     "mozilla/reject-some-requires": [
       "error",
-      "^(chrome|chrome:.*|resource:.*|devtools/server/.*|.*\\.jsm|devtools/shared/platform/(chome|content)/.*)$",
+      "^(chrome|chrome:.*|resource:.*|devtools/server/.*|.*\\.jsm)$",
     ],
   },
 };

devtools/client/shared/sourceeditor/.eslintrc.js

@@ -9,10 +9,10 @@ module.exports = {
   extends: "../../../.eslintrc.js",
 
   rules: {
-    // The inspector is being migrated to HTML and cleaned of
-    // chrome-privileged code, so this rule disallows requiring chrome
-    // code. Some files here disable this rule still. The
-    // goal is to enable the rule globally on all files.
+    // This rule was introduced for the DevTools HTML initiative, the goal was
+    // to avoid requiring helpers unavailable in a regular content page, to
+    // use DevTools as a regular webapplication. Should be reviewed in
+    // Bug 1591091.
     "mozilla/reject-some-requires": [
       "error",
       "^(chrome|chrome:.*|resource:.*|devtools/server/.*|.*\\.jsm)$",

testing/talos/talos/tests/devtools/addon/content/tests/.eslintrc.js

@@ -19,13 +19,6 @@ module.exports = {
     "mozilla/no-aArgs": "error",
     // See bug 1224289.
     "mozilla/reject-importGlobalProperties": ["error", "everything"],
-    // devtools/shared/platform is special; see the README.md in that
-    // directory for details.  We reject requires using explicit
-    // subdirectories of this directory.
-    "mozilla/reject-some-requires": [
-      "error",
-      "^devtools/shared/platform/(chome|content)/",
-    ],
     "mozilla/var-only-at-top-level": "error",
     "mozilla/use-chromeutils-import": ["error", { allowCu: true }],
   },