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

Unified Diff: LayoutTests/inspector/tracing/timeline-js-callstacks.html

Issue 1312563002: DevTools: [timeline tree view] fix total time calculation for recursive case. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: nodesStack -> nodesOnStack Created 5 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 | LayoutTests/inspector/tracing/timeline-js-callstacks-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/inspector/tracing/timeline-js-callstacks.html
diff --git a/LayoutTests/inspector/tracing/timeline-js-callstacks.html b/LayoutTests/inspector/tracing/timeline-js-callstacks.html
index 7bbc09e7fc6889ee9b0210a56a9fa82fbc3ba789..501191e9752668975097ddb4050ccd7c0fa8b81a 100644
--- a/LayoutTests/inspector/tracing/timeline-js-callstacks.html
+++ b/LayoutTests/inspector/tracing/timeline-js-callstacks.html
@@ -323,7 +323,7 @@ function test()
"pid": 17851,
"tid": 23,
"ts": 251000,
- "dur": 8000
+ "dur": 1000
},
{
"args": {
@@ -412,6 +412,131 @@ function test()
"ts": 251400,
},
{
+ "args": {
+ "data": {
+ "stackTrace": [
+ { "functionName": "recursive_b", "scriptId": 1 },
+ { "functionName": "recursive_a", "scriptId": 1 },
+ { "functionName": "recursive_b", "scriptId": 1 },
+ { "functionName": "recursive_a", "scriptId": 1 }
+ ]
+ }
+ },
+ "cat": "disabled-by-default-devtools.timeline",
+ "name": "JSSample",
+ "ph": "I",
+ "pid": 17851,
+ "tid": 23,
+ "ts": 253000,
+ },
+ {
+ "args": {
+ "data": {
+ "stackTrace": [
+ { "functionName": "recursive_a", "scriptId": 1 },
+ { "functionName": "recursive_b", "scriptId": 1 },
+ { "functionName": "recursive_a", "scriptId": 1 }
+ ]
+ }
+ },
+ "cat": "disabled-by-default-devtools.timeline",
+ "name": "JSSample",
+ "ph": "I",
+ "pid": 17851,
+ "tid": 23,
+ "ts": 253008,
+ },
+ {
+ "args": {
+ "data": {
+ "stackTrace": [
+ { "functionName": "recursive_b", "scriptId": 1 },
+ { "functionName": "recursive_a", "scriptId": 1 }
+ ]
+ }
+ },
+ "cat": "disabled-by-default-devtools.timeline",
+ "name": "JSSample",
+ "ph": "I",
+ "pid": 17851,
+ "tid": 23,
+ "ts": 253012,
+ },
+ {
+ "args": {
+ "data": {
+ "stackTrace": [
+ { "functionName": "recursive_a", "scriptId": 1 }
+ ]
+ }
+ },
+ "cat": "disabled-by-default-devtools.timeline",
+ "name": "JSSample",
+ "ph": "I",
+ "pid": 17851,
+ "tid": 23,
+ "ts": 253014,
+ },
+ {
+ "args": {
+ "data": {
+ "stackTrace": [
+ ]
+ }
+ },
+ "cat": "disabled-by-default-devtools.timeline",
+ "name": "JSSample",
+ "ph": "I",
+ "pid": 17851,
+ "tid": 23,
+ "ts": 253015,
+ },
+ {
+ "args": {
+ "data": {
+ "stackTrace": [
+ { "functionName": "recursive_b", "scriptId": 1 },
+ { "functionName": "recursive_a", "scriptId": 1 }
+ ]
+ }
+ },
+ "cat": "disabled-by-default-devtools.timeline",
+ "name": "JSSample",
+ "ph": "I",
+ "pid": 17851,
+ "tid": 23,
+ "ts": 253100,
+ },
+ {
+ "args": {
+ "data": {
+ "stackTrace": [
+ { "functionName": "recursive_a", "scriptId": 1 }
+ ]
+ }
+ },
+ "cat": "disabled-by-default-devtools.timeline",
+ "name": "JSSample",
+ "ph": "I",
+ "pid": 17851,
+ "tid": 23,
+ "ts": 253200,
+ },
+ {
+ "args": {
+ "data": {
+ "stackTrace": [
+ ]
+ }
+ },
+ "cat": "disabled-by-default-devtools.timeline",
+ "name": "JSSample",
+ "ph": "I",
+ "pid": 17851,
+ "tid": 23,
+ "ts": 253300,
+ },
+ {
"args": {},
"cat": "disabled-by-default-devtools.timeline",
"name": "Program",
« no previous file with comments | « no previous file | LayoutTests/inspector/tracing/timeline-js-callstacks-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698