Issue 5535 - Replace our module system with webpack

webpack.config.js

+/*
+ * This file is part of Adblock Plus <https://adblockplus.org/>,
+ * Copyright (C) 2006-present eyeo GmbH
+ *
+ * Adblock Plus is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * Adblock Plus is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+const path = require("path");
+
+let {EXTENSION_PATH, ENTRY_POINTS, OUTPUT_PATH, BUNDLE_NAME,
+     RESOLVE_PATHS, INFO_PATH} = require("process").env;
+
+module.exports = {
+  context: EXTENSION_PATH,
+  devtool: "source-map",

[Wladimir Palant, 2017/09/26 12:06:14]

We might want to use "inline-source-map" here, will make things simpler because
we have only one output file per bundle.

[kzar, 2017/10/02 18:48:32]

It looks like the devtool setting I was using previously was the source of my
eval problems. When I changed to use "source-map" things started behaving!
Done.

+  entry: ENTRY_POINTS.split(" "),
+  output: {
+    path: OUTPUT_PATH,
+    filename: BUNDLE_NAME
+  },
+  resolve: {
+    modules: RESOLVE_PATHS.split(" "),
+    alias: {
+      info$: INFO_PATH,
+      // Prevent builtin Node.js modules from being used instead of our own when
+      // the names clash. Once relative paths are used this won't be necessary.
+      url$: "url.js",
+      events$: "events.js",
+      punycode$: "punycode.js"
+    },
+    plugins: [
+      function()
+      {
+        // Our old module system in packagerChrome.py used to prefix
+        // module names with the name of their parent directory and an
+        // underscore - but only when that directory wasn't called
+        // "lib". This plugin is for backwards compatability, but can
+        // be removed once use of that deprecated syntax has been
+        // replaced.
+        this.plugin("described-resolve", (request, callback) =>
+        {
+          let target = request.request;
+
+          let prefix_index = target.indexOf("_");
+          if (prefix_index == -1)
+            return callback();
+
+          let prefix = target.substring(0, prefix_index);
+          if (prefix == "lib")
+            return callback();
+
+          let fixed_target = path.join(prefix,
+                                       target.substring(prefix_index + 1));
+          return this.doResolve(
+            "resolve", Object.assign({}, request, {request: fixed_target}),
+            "Changed prefixed path using legacy buildtools syntax from " +
+            target + " to " + fixed_target,
+            callback
+          );
+        });
+      }
+    ]
+  },
+  stats: {
+    assets: false,
+    children: false,
+    chunks: false,
+    errorDetails: true,
+    errors: true,
+    hash: false,
+    modules: false,
+    publicPath: false,
+    reasons: false,
+    source: false,
+    timings: false,
+    version: false,
+    warnings: true
+  }
+};