Merge pull request webpack#6227 from EugeneHlushko/feature/sourcemapdevtoolplugin-progress

sokra · web-flow · commit b33bead87649 · 2018-01-04T21:43:50.000+01:00
Feature: SourceMapDevToolPlugin progress: wip
diff --git a/lib/SourceMapDevToolPlugin.js b/lib/SourceMapDevToolPlugin.js
@@ -82,39 +82,58 @@ class SourceMapDevToolPlugin {
 		compiler.hooks.compilation.tap("SourceMapDevToolPlugin", (compilation) => {
 			new SourceMapDevToolModuleOptionsPlugin(options).apply(compilation);
 
-			compilation.hooks.afterOptimizeChunkAssets.tap("SourceMapDevToolPlugin", chunks => {
+			compilation.hooks.afterOptimizeChunkAssets.tap({
+				name: "SourceMapDevToolPlugin",
+				context: true
+			}, (context, chunks) => {
 				const moduleToSourceNameMapping = new Map();
-				const tasks = [];
+				const reportProgress = (context && context.reportProgress) ? context.reportProgress : () => {};
 
+				const files = [];
 				chunks.forEach(chunk => {
 					chunk.files.forEach(file => {
 						if(matchObject(file)) {
-							const task = getTaskForFile(file, chunk, options, compilation);
-
-							if(task) {
-								const modules = task.sourceMap.sources.map(source => {
-									const module = compilation.findModule(source);
-									return module || source;
-								});
-
-								for(let idx = 0; idx < modules.length; idx++) {
-									const module = modules[idx];
-									if(!moduleToSourceNameMapping.get(module)) {
-										moduleToSourceNameMapping.set(module, ModuleFilenameHelpers.createFilename(module, {
-											moduleFilenameTemplate: moduleFilenameTemplate,
-											namespace: namespace
-										}, requestShortener));
-									}
-								}
-
-								task.modules = modules;
-
-								tasks.push(task);
-							}
+							files.push({
+								file,
+								chunk
+							});
 						}
 					});
 				});
 
+				reportProgress(0.0);
+				const tasks = [];
+				files.forEach(({
+					file,
+					chunk
+				}, idx) => {
+					reportProgress(0.5 * idx / files.length, file, "generate SourceMap");
+					const task = getTaskForFile(file, chunk, options, compilation);
+
+					if(task) {
+						const modules = task.sourceMap.sources.map(source => {
+							const module = compilation.findModule(source);
+							return module || source;
+						});
+
+						for(let idx = 0; idx < modules.length; idx++) {
+							const module = modules[idx];
+							if(!moduleToSourceNameMapping.get(module)) {
+								moduleToSourceNameMapping.set(module, ModuleFilenameHelpers.createFilename(module, {
+									moduleFilenameTemplate: moduleFilenameTemplate,
+									namespace: namespace
+								}, requestShortener));
+							}
+						}
+
+						task.modules = modules;
+
+						tasks.push(task);
+					}
+
+				});
+
+				reportProgress(0.5, "resolve sources");
 				const usedNamesSet = new Set(moduleToSourceNameMapping.values());
 				const conflictDetectionSet = new Set();
 
@@ -155,7 +174,8 @@ class SourceMapDevToolPlugin {
 					moduleToSourceNameMapping.set(module, sourceName);
 					usedNamesSet.add(sourceName);
 				}
-				tasks.forEach(task => {
+				tasks.forEach((task, index) => {
+					reportProgress(0.5 + 0.5 * index / tasks.length, task.file, "attach SourceMap");
 					const chunk = task.chunk;
 					const file = task.file;
 					const asset = task.asset;
@@ -208,6 +228,7 @@ class SourceMapDevToolPlugin {
 						);
 					}
 				});
+				reportProgress(1.0);
 			});
 		});
 	}
