Breaking: Avoid loading sources during the write method

Author: phated

lib/helpers.js

@@ -214,31 +214,6 @@ function commentFormatter(extname, url) {
   return convert.generateMapFileComment(url, opts);
 }
 
-function includeContent(file, state, callback) {
-
-  state.sourceMap.sourcesContent = state.sourceMap.sourcesContent || [];
-
-  function loadSources(sourcePath, idx, cb) {
-    if (state.sourceMap.sourcesContent[idx]) {
-      return cb();
-    }
-
-    var basePath = state.sourceMap.sourceRoot || file.base;
-    var absPath = path.resolve(basePath, sourcePath);
-    fs.readFile(absPath, 'utf8', onRead);
-
-    function onRead(err, data) {
-      if (err) {
-        return cb();
-      }
-      state.sourceMap.sourcesContent[idx] = stripBom(data);
-      cb();
-    }
-  }
-
-  async.eachOf(file.sourceMap.sources, loadSources, callback);
-}
-
 function contentIncluded(file, state, callback) {
 
   var comment;
@@ -277,7 +252,6 @@ function contentIncluded(file, state, callback) {
 
 function writeSourceMaps(file, state, callback) {
   var tasks = [
-    includeContent,
     contentIncluded
   ];
 

test/write.js

@@ -243,25 +243,4 @@ describe('write', function() {
 			done(err);
 		});
 	});
-
-	it('should fetch missing sourceContent', function(done) {
-		var file = makeFile();
-		delete file.sourceMap.sourcesContent;
-		sourcemaps.write(file, function(err, updatedFile) {
-			expect(updatedFile.sourceMap.sourcesContent).toNotBe(undefined);
-			expect(updatedFile.sourceMap.sourcesContent).toEqual([sourceContent]);
-			done(err);
-		});
-	});
-
-	it('should not throw when unable to fetch missing sourceContent', function(done) {
-		var file = makeFile();
-		file.sourceMap.sources[0] += '.invalid';
-		delete file.sourceMap.sourcesContent;
-		sourcemaps.write(file, function(err, updatedFile) {
-			expect(updatedFile.sourceMap.sourcesContent).toNotBe(undefined);
-			expect(updatedFile.sourceMap.sourcesContent).toEqual([]);
-			done(err);
-		});
-	});
 });