Fix the case where the generated source maps contains a null source c…

…ontent

minifier.js

@@ -59,6 +59,8 @@ module.exports = function (opts, uglify) {
       options.outSourceMap = file.relative;
     }
 
+    var originalContents = String(file.contents);
+
     var mangled = trycatch(function () {
       var m = uglify.minify(String(file.contents), options);
       m.code = new Buffer(m.code.replace(reSourceMapComment, ''));
@@ -74,6 +76,7 @@ module.exports = function (opts, uglify) {
     if (file.sourceMap) {
       var sourceMap = JSON.parse(mangled.map);
       sourceMap.sources = [file.relative];
+      sourceMap.sourcesContent = [originalContents];
       applySourceMap(file, sourceMap);
     }
 

test/sourcemap.js

@@ -158,3 +158,50 @@ test('should not remember source maps across files', function (t) {
   mangled.write(testFile2);
   mangled.end();
 });
+
+test('should avoid "ghost" files in sourcemaps', function (t) {
+  t.plan(14);
+
+  var testBabelInput = '"use strict";\n\n(function (first, second) {\n    console.log(first + second);\n})(5, 10);';
+  var testBabelExpected = uglifyjs.minify(testBabelInput, {fromString: true}).code;
+
+  var testFile1 = new Vinyl({
+    cwd: '/home/terin/broken-promises/',
+    base: '/home/terin/broken-promises/test',
+    path: '/home/terin/broken-promises/test/all.js',
+    contents: new Buffer(testBabelInput)
+  });
+  testFile1.sourceMap = {
+    version: 3,
+    file: 'all.js',
+    sources: ['test1.js'],
+    names: [],
+    mappings: ';;AAAA,AAAC,CAAA,UAAS,KAAK,EAAE,MAAM,EAAE;AACrB,WAAO,CAAC,GAAG,CAAC,KAAK,GAAG,MAAM,CAAC,CAAC;CAC/B,CAAA,CAAC,CAAC,EAAE,EAAE,CAAC,CAAE',
+    sourcesContent: [testContents1Input]
+  };
+
+  var mangled = gulpUglify();
+
+  mangled.on('data', function (newFile) {
+    t.ok(newFile, 'emits a file');
+    t.ok(newFile.path, 'file has a path');
+    t.ok(newFile.relative, 'file has relative path information');
+    t.ok(newFile.contents, 'file has contents');
+
+    t.ok(newFile.contents instanceof Buffer, 'file contents are a buffer');
+
+    t.equals(String(newFile.contents), testBabelExpected);
+
+    t.ok(newFile.sourceMap, 'has a source map');
+    t.equals(newFile.sourceMap.version, 3, 'source map has expected version');
+    t.ok(Array.isArray(newFile.sourceMap.sources), 'source map has sources array');
+    t.deepEquals(newFile.sourceMap.sources, ['all.js', 'test1.js'], 'sources array has the inputs');
+    t.ok(Array.isArray(newFile.sourceMap.names), 'source maps has names array');
+    t.ok(newFile.sourceMap.mappings, 'source map has mappings');
+    t.ok(Array.isArray(newFile.sourceMap.sourcesContent), 'source maps has sources content array');
+    t.deepEquals(newFile.sourceMap.sourcesContent, [testBabelInput, testContents1Input], 'sources array has the inputs');
+  });
+
+  mangled.write(testFile1);
+  mangled.end();
+});