Bug 1312041 - make loader-plugin-raw.jsm eslint-clean; r=jryans
☠☠ backed out by 8dcdf56296d5 ☠ ☠
authorTom Tromey <tom@tromey.com>
Tue, 25 Oct 2016 21:24:13 -0600
changeset 347114 9bfbc8480d437b2fe843ee50f2e22b7b73545cc5
parent 347113 bd45588af02e58ab94344d352e8bd09d4125d06b
child 347115 c4f42d7d5453c10bbc9f90ed121210ea0eda76d4
push id10298
push userraliiev@mozilla.com
push dateMon, 14 Nov 2016 12:33:03 +0000
treeherdermozilla-aurora@7e29173b1641 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjryans
bugs1312041
milestone52.0a1
Bug 1312041 - make loader-plugin-raw.jsm eslint-clean; r=jryans MozReview-Commit-ID: 4Lkua3ZMslL
.eslintignore
devtools/shared/loader-plugin-raw.jsm
--- a/.eslintignore
+++ b/.eslintignore
@@ -129,18 +129,16 @@ devtools/server/tests/**
 devtools/shared/*.js
 !devtools/shared/async-storage.js
 !devtools/shared/async-utils.js
 !devtools/shared/defer.js
 !devtools/shared/event-emitter.js
 !devtools/shared/indentation.js
 !devtools/shared/loader-plugin-raw.jsm
 !devtools/shared/task.js
-devtools/shared/*.jsm
-!devtools/shared/Loader.jsm
 devtools/shared/apps/**
 devtools/shared/client/**
 devtools/shared/discovery/**
 devtools/shared/gcli/**
 !devtools/shared/gcli/templater.js
 devtools/shared/heapsnapshot/**
 devtools/shared/layout/**
 devtools/shared/locales/**
--- a/devtools/shared/loader-plugin-raw.jsm
+++ b/devtools/shared/loader-plugin-raw.jsm
@@ -8,41 +8,40 @@ const { utils: Cu } = Components;
 const { NetUtil } = Cu.import("resource://gre/modules/NetUtil.jsm", {});
 
 /**
  * A function that can be used as part of a require hook for a
  * loader.js Loader.  This function only handles webpack-style "raw!"
  * requires; other requires should not be passed to this.  See
  * https://github.com/webpack/raw-loader.
  */
-function requireRawId(id, require) {
+this.requireRawId = function (id, require) {
   // Add the chrome:// protocol for properties files if missing (see Bug 1294220)
   if (id.endsWith(".properties") && !id.startsWith("raw!chrome://")) {
     id = id.replace("raw!", "raw!chrome://");
   }
 
   let uri = require.resolve(id.slice(4));
   // If the original string did not end with ".js", then
   // require.resolve might have added the suffix.  We don't want to
   // add a suffix for a raw load (if needed the caller can specify it
   // manually), so remove it here.
   if (!id.endsWith(".js") && uri.endsWith(".js")) {
     uri = uri.slice(0, -3);
   }
 
-
   let stream = NetUtil.newChannel({
     uri: NetUtil.newURI(uri, "UTF-8"),
     loadUsingSystemPrincipal: true
   }).open2();
 
   let count = stream.available();
   let data = NetUtil.readInputStreamToString(stream, count, {
     charset: "UTF-8"
   });
   stream.close();
 
   // For the time being it doesn't seem worthwhile to cache the
   // result here.
   return data;
-}
+};
 
 this.EXPORTED_SYMBOLS = ["requireRawId"];