Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(242)

Unified Diff: third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping.html

Issue 2222713004: [devtools] _parseMap in SourceMap.js does not strictly follow specs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: [devtools] _parseMap in SourceMap.js does not strictly follow specs Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « AUTHORS ('k') | third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping.html b/third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping.html
index fa8b45ea8e018b24244f88a3b46b0aa8f0e06a98..528c6afab6237c784a507e471d1dfd01e07c66a8 100644
--- a/third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping.html
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping.html
@@ -475,6 +475,50 @@ function test()
checkMapping(0, 9, "example.js", 0, 9, mapping);
checkReverseMapping(0, 0, "example.js", 0, mapping);
next();
+ },
+
+ function testNameIndexes(next)
+ {
+ /*
+ ------------------------------------------------------------------------------------
+ chrome_issue_611738.clj:
+ (ns devtools-sample.chrome-issue-611738)
+
+ (defmacro m []
+ `(let [generated# "value2"]))
+ ------------------------------------------------------------------------------------
+ chrome_issue_611738.cljs:
+ (ns devtools-sample.chrome-issue-611738
+ (:require-macros [devtools-sample.chrome-issue-611738 :refer [m]]))
+
+ (let [name1 "value1"]
+ (m))
+ ------------------------------------------------------------------------------------
+ chrome_issue_611738.js:
+ // Compiled by ClojureScript 1.9.89 {}
+ goog.provide('devtools_sample.chrome_issue_611738');
+ goog.require('cljs.core');
+ var name1_31466 = "value1";
+ var generated31465_31467 = "value2";
+
+ //# sourceMappingURL=chrome_issue_611738.js.map
+ ------------------------------------------------------------------------------------
+ chrome_issue_611738.js.map:
+ {"version":3,"file":"\/Users\/darwin\/code\/cljs-devtools-sample\/resources\/public\/_compiled\/demo\/devtools_sample\/chrome_issue_611738.js","sources":["chrome_issue_611738.cljs"],"lineCount":7,"mappings":";AAAA;;AAGA,kBAAA,dAAMA;AAAN,AACE,IAAAC,uBAAA;AAAA,AAAA","names":["name1","generated31465"]}
+ ------------------------------------------------------------------------------------
+ */
+
+ var mappingPayload = {
+ "sources": ["chrome_issue_611738.cljs"],
+ "mappings": ";AAAA;;AAGA,kBAAA,dAAMA;AAAN,AACE,IAAAC,uBAAA;AAAA,AAAA",
+ "names": ["name1", "generated31465"]
+ };
+ var mapping = new WebInspector.TextSourceMap("chrome_issue_611738.js", "chrome_issue_611738.js.map", mappingPayload);
+ mapping.mappings().forEach(function(entry) {
+ const name = entry.name ? "'" + entry.name + "'" : "[no name assigned]";
+ InspectorTest.addResult(entry.lineNumber + ":" + entry.columnNumber + " > " + name);
+ });
+ next();
}
]);
};
« no previous file with comments | « AUTHORS ('k') | third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698