only generate sourcemap if needed

Author: yyx990803

lib/loader.js

@@ -135,7 +135,7 @@ module.exports = function (content) {
     }
   }
 
-  var parts = parse(content, fileName)
+  var parts = parse(content, fileName, this.sourceMap)
   var hasLocalStyles = false
   var output = 'var __vue_script__, __vue_template__\n'
 

lib/parser.js

@@ -7,7 +7,7 @@ var deindent = require('de-indent')
 var splitRE = /\r?\n/g
 var emptyRE = /^\s*$/
 
-module.exports = function (content, filename) {
+module.exports = function (content, filename, needMap) {
 
   var cacheKey = hash(filename + content)
   // source-map cache busting for hot-reloadded modules
@@ -32,6 +32,7 @@ module.exports = function (content, filename) {
     var src = getAttribute(node, 'src')
     var scoped = getAttribute(node, 'scoped') != null
     var warnings = null
+    var map = null
 
     if (!output[type]) {
       return
@@ -101,36 +102,37 @@ module.exports = function (content, filename) {
       result = deindent(content.slice(start, end))
     }
 
-    // generate source map
-    var map = new SourceMapGenerator()
-    map.setSourceContent(filenameWithHash, content)
-    result.split(splitRE).forEach(function (line, index) {
-      map.addMapping({
-        source: filenameWithHash,
-        original: {
-          line: index + 1 + lineOffset,
-          column: 0
-        },
-        generated: {
-          line: index + 1,
-          column: 0
-        }
+    if (needMap) {
+      // generate source map
+      map = new SourceMapGenerator()
+      map.setSourceContent(filenameWithHash, content)
+      result.split(splitRE).forEach(function (line, index) {
+        map.addMapping({
+          source: filenameWithHash,
+          original: {
+            line: index + 1 + lineOffset,
+            column: 0
+          },
+          generated: {
+            line: index + 1,
+            column: 0
+          }
+        })
       })
-    })
-
-    // workaround for Webpack eval-source-map bug
-    // https://github.com/webpack/webpack/pull/1816
-    // in case the script was piped through another loader
-    // that doesn't pass down the source map properly.
-    if (type === 'script' && !lang) {
-      result += '\n/* generated by vue-loader */\n'
+      // workaround for Webpack eval-source-map bug
+      // https://github.com/webpack/webpack/pull/1816
+      // in case the script was piped through another loader
+      // that doesn't pass down the source map properly.
+      if (type === 'script' && !lang) {
+        result += '\n/* generated by vue-loader */\n'
+      }
     }
 
     output[type].push({
       lang: lang,
       scoped: scoped,
       content: result,
-      map: map.toJSON(),
+      map: map && map.toJSON(),
       warnings: warnings
     })
   })

lib/selector.js

@@ -6,7 +6,7 @@ module.exports = function (content) {
   this.cacheable()
   var query = loaderUtils.parseQuery(this.query)
   var filename = path.basename(this.resourcePath)
-  var parts = parse(content, filename)
+  var parts = parse(content, filename, this.sourceMap)
   var part = parts[query.type][query.index]
   this.callback(null, part.content, part.map)
 }