Bug 1392602 - Instanciate RegExp only once in DevTools loader. r=jdescottes draft
authorAlexandre Poirot <poirot.alex@gmail.com>
Mon, 28 Aug 2017 14:05:17 +0200
changeset 658607 3ed4742d60de7d8911b21e3b094f1d3cc1ccf67e
parent 658606 ad13cf51f448aa6a3db02152e483a4840e28c45d
child 658608 74e9d76a9f3357033cafdac573a35dcac4144f6e
push id77824
push userbmo:poirot.alex@gmail.com
push dateMon, 04 Sep 2017 12:19:59 +0000
reviewersjdescottes
bugs1392602
milestone57.0a1
Bug 1392602 - Instanciate RegExp only once in DevTools loader. r=jdescottes MozReview-Commit-ID: GGYMPKZwBsS
devtools/shared/base-loader.js
--- a/devtools/shared/base-loader.js
+++ b/devtools/shared/base-loader.js
@@ -759,15 +759,17 @@ function Loader(options) {
     load: { enumerable: false, value: options.load || load },
     requireHook: { enumerable: false, value: options.requireHook },
     loadModuleHook: { enumerable: false, value: options.loadModuleHook },
   };
 
   return Object.create(null, returnObj);
 };
 
-var isSystemURI = uri => /^resource:\/\/(gre|devtools|testing-common)\//.test(uri);
+var SystemRegExp = /^resource:\/\/(gre|devtools|testing-common)\//;
+var isSystemURI = uri => SystemRegExp.test(uri);
 
 var isJSONURI = uri => uri.endsWith('.json');
 var isJSMURI = uri => uri.endsWith('.jsm');
 var isJSURI = uri => uri.endsWith('.js');
-var isAbsoluteURI = uri => /^(resource|chrome|file|jar):/.test(uri);
+var AbsoluteRegExp = /^(resource|chrome|file|jar):/;
+var isAbsoluteURI = uri => AbsoluteRegExp.test(uri);
 var isRelative = id => id.startsWith(".");