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

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: 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 | « no previous file | 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..ea9384239aca84000cb3908ce2f20d919517f069 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,47 @@ function test()
checkMapping(0, 9, "example.js", 0, 9, mapping);
checkReverseMapping(0, 0, "example.js", 0, mapping);
next();
+ },
+
+ function testIssue611738(next)
lushnikov 2016/08/25 19:48:36 let's name this testNameIndexes
+ {
+ /*
+ ------------------------------------------------------------------------------------
+ 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"]}
+ ------------------------------------------------------------------------------------
+ Related links:
lushnikov 2016/08/25 19:48:36 let's remove the "related links"
+ https://gist.github.com/darwin/4103df378bc21ed0af41897e2f5b1762
+ https://sokra.github.io/source-map-visualization/#base64,Ly8gQ29tcGlsZWQgYnkgQ2xvanVyZVNjcmlwdCAxLjkuODkge30KZ29vZy5wcm92aWRlKCdkZXZ0b29sc19zYW1wbGUuY2hyb21lX2lzc3VlXzYxMTczOCcpOwpnb29nLnJlcXVpcmUoJ2NsanMuY29yZScpOwp2YXIgbmFtZTFfMzE0NjYgPSAidmFsdWUxIjsKdmFyIGdlbmVyYXRlZDMxNDY1XzMxNDY3ID0gInZhbHVlMiI7CgovLyMgc291cmNlTWFwcGluZ1VSTD1jaHJvbWVfaXNzdWVfNjExNzM4LmpzLm1hcAo=,eyJ2ZXJzaW9uIjozLCJmaWxlIjoiL1VzZXJzL2Rhcndpbi9jb2RlL2NsanMtZGV2dG9vbHMtc2FtcGxlL3Jlc291cmNlcy9wdWJsaWMvX2NvbXBpbGVkL2RlbW8vZGV2dG9vbHNfc2FtcGxlL2Nocm9tZV9pc3N1ZV82MTE3MzguanMiLCJzb3VyY2VzIjpbImNocm9tZV9pc3N1ZV82MTE3MzguY2xqcyJdLCJsaW5lQ291bnQiOjcsIm1hcHBpbmdzIjoiO0FBQUE7O0FBR0Esa0JBQUEsZEFBTUE7QUFBTixBQUNFLElBQUFDLHVCQUFBO0FBQUEsQUFBQSIsIm5hbWVzIjpbIm5hbWUxIiwiZ2VuZXJhdGVkMzE0NjUiXX0=,KG5zIGRldnRvb2xzLXNhbXBsZS5jaHJvbWUtaXNzdWUtNjExNzM4CiAgKDpyZXF1aXJlLW1hY3JvcyBbZGV2dG9vbHMtc2FtcGxlLmNocm9tZS1pc3N1ZS02MTE3MzggOnJlZmVyIFttXV0pKQoKKGxldCBbbmFtZTEgInZhbHVlMSJdCiAgKG0pKQoK
+ */
+
+ 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 | « no previous file | 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