Bug 1586757 - Rename .jsm files to .jsm.js to allow type checking; r=julienw

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

--HG--
rename : devtools/client/performance-new/popup/background.jsm => devtools/client/performance-new/popup/background.jsm.js
rename : devtools/client/performance-new/popup/menu-button.jsm => devtools/client/performance-new/popup/menu-button.jsm.js
extra : moz-landing-system : lando

devtools/client/menus.js

@@ -64,7 +64,7 @@ loader.lazyImporter(
 loader.lazyImporter(
   this,
   "ProfilerMenuButton",
-  "resource://devtools/client/performance-new/popup/menu-button.jsm"
+  "resource://devtools/client/performance-new/popup/menu-button.jsm.js"
 );
 loader.lazyRequireGetter(
   this,

devtools/client/performance-new/popup/initializer.js

@@ -35,7 +35,7 @@ const {
   setRecordingPreferencesOnBrowser,
   getSymbolsFromThisBrowser,
 } = ChromeUtils.import(
-  "resource://devtools/client/performance-new/popup/background.jsm"
+  "resource://devtools/client/performance-new/popup/background.jsm.js"
 );
 
 const { receiveProfile } = require("devtools/client/performance-new/browser");

devtools/client/performance-new/popup/moz.build

@@ -4,8 +4,8 @@
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 DevToolsModules(
-    'background.jsm',
-    'menu-button.jsm',
+    'background.jsm.js',
+    'menu-button.jsm.js',
 )
 
 with Files('**'):

devtools/client/performance-new/test/browser/head.js

@@ -108,7 +108,7 @@ async function makeSureProfilerPopupIsEnabled() {
 
   info("> Load the profiler menu button.");
   const { ProfilerMenuButton } = ChromeUtils.import(
-    "resource://devtools/client/performance-new/popup/menu-button.jsm"
+    "resource://devtools/client/performance-new/popup/menu-button.jsm.js"
   );
 
   if (!ProfilerMenuButton.isEnabled()) {

devtools/client/performance-new/test/xpcshell/test_popup_initial_state.js

@@ -9,7 +9,7 @@
 
 function setupBackgroundJsm() {
   const background = ChromeUtils.import(
-    "resource://devtools/client/performance-new/popup/background.jsm"
+    "resource://devtools/client/performance-new/popup/background.jsm.js"
   );
   return background;
 }

devtools/startup/DevToolsStartup.jsm

@@ -64,7 +64,7 @@ ChromeUtils.defineModuleGetter(
 ChromeUtils.defineModuleGetter(
   this,
   "ProfilerMenuButton",
-  "resource://devtools/client/performance-new/popup/menu-button.jsm"
+  "resource://devtools/client/performance-new/popup/menu-button.jsm.js"
 );
 
 // We don't want to spend time initializing the full loader here so we create
@@ -259,7 +259,7 @@ function isProfilerButtonEnabled() {
 
 XPCOMUtils.defineLazyGetter(this, "ProfilerPopupBackground", function() {
   return ChromeUtils.import(
-    "resource://devtools/client/performance-new/popup/background.jsm"
+    "resource://devtools/client/performance-new/popup/background.jsm.js"
   );
 });
 

tools/lint/eslint/eslint-plugin-mozilla/lib/configs/recommended.js

@@ -65,7 +65,7 @@ module.exports = {
         browser: false,
         "mozilla/jsm": true,
       },
-      files: "**/*.jsm",
+      files: ["**/*.jsm", "**/*.jsm.js"],
       rules: {
         "mozilla/mark-exported-symbols-as-used": "error",
         // TODO: Bug 1575506 turn `builtinGlobals` on here.