Add //# sourceURL along with source map.

dtinth · dtinth · commit c3902570ac79 · 2014-12-04T12:11:15.000+07:00
For some odd reason, Google Chrome requires sourceURL to be able to
display source maps in stack traces correctly.
diff --git a/src/loader.js b/src/loader.js
@@ -1103,8 +1103,10 @@ function logloads(loads) {
 
       var sourceMap = compiler.getSourceMap();
 
-      if (__global.btoa && sourceMap)
+      if (__global.btoa && sourceMap) {
+        source += '\n//# sourceURL=' + load.address + '!eval';
         source += '\n//# sourceMappingURL=data:application/json;base64,' + btoa(unescape(encodeURIComponent(sourceMap))) + '\n';
+      }
 
       source = 'var __moduleAddress = "' + load.address + '";' + source;
 
diff --git a/test/manual/source-maps/test.html b/test/manual/source-maps/test.html
@@ -0,0 +1,13 @@
+<!doctype html>
+  <html>
+
+  <body>
+    <h1>Source Map Stack Trace Test</h1>
+    <script src="../../../node_modules/traceur/bin/traceur.js"></script>
+    <script src="../../../dist/es6-module-loader.src.js"></script>
+    <script>
+      System.import('test').then(function(module) {
+        console.log(module.make().stack)
+      })
+    </script>
+  </body>
diff --git a/test/manual/source-maps/test.js b/test/manual/source-maps/test.js
@@ -0,0 +1,4 @@
+export function make() {
+  return new Error('Hello')
+}
+

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +export function make() {
 +  return new Error('Hello')
 +}
++