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

Side by Side Diff: third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js

Issue 2169153002: DevTools: make all timeline trace events use 0-based line numbers (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update expectation Created 4 years, 5 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 | third_party/WebKit/LayoutTests/inspector/tracing/timeline-js/timeline-script-id-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 function wrapCallFunctionForTimeline(f) 1 function wrapCallFunctionForTimeline(f)
2 { 2 {
3 var script = document.createElement("script"); 3 var script = document.createElement("script");
4 script.textContent = "(" + f.toString() + ")()\n//# sourceURL=wrapCallFuncti onForTimeline.js"; 4 script.textContent = "(" + f.toString() + ")()\n//# sourceURL=wrapCallFuncti onForTimeline.js";
5 document.body.appendChild(script); 5 document.body.appendChild(script);
6 } 6 }
7 7
8 var initialize_Timeline = function() { 8 var initialize_Timeline = function() {
9 9
10 InspectorTest.preloadPanel("timeline"); 10 InspectorTest.preloadPanel("timeline");
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 paintId: "skip", 57 paintId: "skip",
58 startTime: "skip", 58 startTime: "skip",
59 }; 59 };
60 60
61 InspectorTest.formatters.formatAsInvalidationCause = function(cause) 61 InspectorTest.formatters.formatAsInvalidationCause = function(cause)
62 { 62 {
63 if (!cause) 63 if (!cause)
64 return "<undefined>"; 64 return "<undefined>";
65 var stackTrace; 65 var stackTrace;
66 if (cause.stackTrace && cause.stackTrace.length) 66 if (cause.stackTrace && cause.stackTrace.length)
67 stackTrace = InspectorTest.formatters.formatAsURL(cause.stackTrace[0].ur l) + ":" + cause.stackTrace[0].lineNumber; 67 stackTrace = InspectorTest.formatters.formatAsURL(cause.stackTrace[0].ur l) + ":" + (cause.stackTrace[0].lineNumber + 1);
68 return "{reason: " + cause.reason + ", stackTrace: " + stackTrace + "}"; 68 return "{reason: " + cause.reason + ", stackTrace: " + stackTrace + "}";
69 } 69 }
70 70
71 InspectorTest.preloadPanel("timeline"); 71 InspectorTest.preloadPanel("timeline");
72 WebInspector.TempFile = InspectorTest.TempFileMock; 72 WebInspector.TempFile = InspectorTest.TempFileMock;
73 73
74 InspectorTest.createTracingModel = function() 74 InspectorTest.createTracingModel = function()
75 { 75 {
76 return new WebInspector.TracingModel(new WebInspector.TempFileBackingStorage ("tracing")); 76 return new WebInspector.TracingModel(new WebInspector.TempFileBackingStorage ("tracing"));
77 } 77 }
(...skipping 392 matching lines...) Expand 10 before | Expand all | Expand 10 after
470 if (!--count) { 470 if (!--count) {
471 callback(); 471 callback();
472 return; 472 return;
473 } 473 }
474 if (window.testRunner) 474 if (window.testRunner)
475 testRunner.capturePixelsAsyncThen(requestAnimationFrame.bind(window, makeFrame)); 475 testRunner.capturePixelsAsyncThen(requestAnimationFrame.bind(window, makeFrame));
476 else 476 else
477 window.requestAnimationFrame(makeFrame); 477 window.requestAnimationFrame(makeFrame);
478 } 478 }
479 } 479 }
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/inspector/tracing/timeline-js/timeline-script-id-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698