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 320421 9bfbc8480d437b2fe843ee50f2e22b7b73545cc5
parent 320420 bd45588af02e58ab94344d352e8bd09d4125d06b
child 320422 c4f42d7d5453c10bbc9f90ed121210ea0eda76d4
push id30901
push userkwierso@gmail.com
push dateThu, 03 Nov 2016 00:16:45 +0000
treeherdermozilla-central@ac55a6776435 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjryans
bugs1312041
milestone52.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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"];