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

Side by Side Diff: src/inspector/js_protocol.json

Issue 2741653003: [inspector] do not translate offset to line/column for coverage. (Closed)
Patch Set: Created 3 years, 9 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 unified diff | Download patch
« no previous file with comments | « no previous file | src/inspector/v8-profiler-agent-impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 { 1 {
2 "version": { "major": "1", "minor": "2" }, 2 "version": { "major": "1", "minor": "2" },
3 "domains": [ 3 "domains": [
4 { 4 {
5 "domain": "Schema", 5 "domain": "Schema",
6 "description": "Provides information about the protocol schema.", 6 "description": "Provides information about the protocol schema.",
7 "types": [ 7 "types": [
8 { 8 {
9 "id": "Domain", 9 "id": "Domain",
10 "type": "object", 10 "type": "object",
(...skipping 830 matching lines...) Expand 10 before | Expand all | Expand 10 after
841 "description": "Specifies a number of samples attributed to a ce rtain source position.", 841 "description": "Specifies a number of samples attributed to a ce rtain source position.",
842 "properties": [ 842 "properties": [
843 { "name": "line", "type": "integer", "description": "Source line number (1-based)." }, 843 { "name": "line", "type": "integer", "description": "Source line number (1-based)." },
844 { "name": "ticks", "type": "integer", "description": "Number of samples attributed to the source line." } 844 { "name": "ticks", "type": "integer", "description": "Number of samples attributed to the source line." }
845 ] 845 ]
846 }, 846 },
847 { "id": "CoverageRange", 847 { "id": "CoverageRange",
848 "type": "object", 848 "type": "object",
849 "description": "Coverage data for a source range.", 849 "description": "Coverage data for a source range.",
850 "properties": [ 850 "properties": [
851 { "name": "startLineNumber", "type": "integer", "description ": "JavaScript script line number (0-based) for the range start." },
pfeldman 2017/03/09 21:18:53 protocol lgtm. we can't change front-end though be
852 { "name": "startColumnNumber", "type": "integer", "descripti on": "JavaScript script column number (0-based) for the range start." },
853 { "name": "endLineNumber", "type": "integer", "description": "JavaScript script line number (0-based) for the range end." },
854 { "name": "endColumnNumber", "type": "integer", "description ": "JavaScript script column number (0-based) for the range end." },
855 { "name": "startOffset", "type": "integer", "description": " JavaScript script source offset for the range start." }, 851 { "name": "startOffset", "type": "integer", "description": " JavaScript script source offset for the range start." },
856 { "name": "endOffset", "type": "integer", "description": "Ja vaScript script source offset for the range end." }, 852 { "name": "endOffset", "type": "integer", "description": "Ja vaScript script source offset for the range end." },
857 { "name": "count", "type": "integer", "description": "Collec ted execution count of the source range." } 853 { "name": "count", "type": "integer", "description": "Collec ted execution count of the source range." }
858 ], 854 ],
859 "experimental": true 855 "experimental": true
860 }, 856 },
861 { "id": "FunctionCoverage", 857 { "id": "FunctionCoverage",
862 "type": "object", 858 "type": "object",
863 "description": "Coverage data for a JavaScript function.", 859 "description": "Coverage data for a JavaScript function.",
864 "properties": [ 860 "properties": [
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
1075 { 1071 {
1076 "name": "heapStatsUpdate", 1072 "name": "heapStatsUpdate",
1077 "description": "If heap objects tracking has been started then b ackend may send update for one or more fragments", 1073 "description": "If heap objects tracking has been started then b ackend may send update for one or more fragments",
1078 "parameters": [ 1074 "parameters": [
1079 { "name": "statsUpdate", "type": "array", "items": { "type": "integer" }, "description": "An array of triplets. Each triplet describes a fra gment. The first integer is the fragment index, the second integer is a total co unt of objects for the fragment, the third integer is a total size of the object s for the fragment."} 1075 { "name": "statsUpdate", "type": "array", "items": { "type": "integer" }, "description": "An array of triplets. Each triplet describes a fra gment. The first integer is the fragment index, the second integer is a total co unt of objects for the fragment, the third integer is a total size of the object s for the fragment."}
1080 ] 1076 ]
1081 } 1077 }
1082 ] 1078 ]
1083 }] 1079 }]
1084 } 1080 }
OLDNEW
« no previous file with comments | « no previous file | src/inspector/v8-profiler-agent-impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698