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

Unified Diff: third_party/WebKit/LayoutTests/inspector/tracing/trace-event-self-time.html

Issue 2262543002: DevTools: Profiler domain refactoring: encode timestamps as deltas. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 4 landing 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
Index: third_party/WebKit/LayoutTests/inspector/tracing/trace-event-self-time.html
diff --git a/third_party/WebKit/LayoutTests/inspector/tracing/trace-event-self-time.html b/third_party/WebKit/LayoutTests/inspector/tracing/trace-event-self-time.html
index 0c008205f5054bc92dea86d27d0a1b9c06e19960..43a165dfd60328a9eef6418a51dee5174f785f92 100644
--- a/third_party/WebKit/LayoutTests/inspector/tracing/trace-event-self-time.html
+++ b/third_party/WebKit/LayoutTests/inspector/tracing/trace-event-self-time.html
@@ -260,8 +260,8 @@ function test()
];
var cpuProfile = {
- startTime: 420,
- endTime: 430,
+ startTime: 420e3,
+ endTime: 430e3,
nodes: [
{
callFrame: { functionName: "(root)" },
@@ -282,8 +282,8 @@ function test()
id: 4
}
],
- timestamps: [421000, 422000, 423000, 424000, 425000, 426000, 427000, 428000, 429000],
- samples: [2, 2, 3, 3, 3, 4, 4, 2, 2 ]
+ timestampDeltas: new Array(9).fill(1000),
+ samples: [2, 2, 3, 3, 3, 4, 4, 2, 2]
};
var timelineController = InspectorTest.timelineController();
@@ -291,7 +291,7 @@ function test()
timelineController.traceEventsCollected(rawTraceEvents);
timelineController._allSourcesFinished();
var events = InspectorTest.timelineModel().inspectedTargetEvents();
- events.forEach(e => InspectorTest.addResult(e.name + ": " + e.startTime + " " + (e.selfTime || 0).toFixed(2) + "/" + (e.duration || 0).toFixed(2)));
+ events.forEach(e => InspectorTest.addResult(`${e.name}: ${e.startTime} ${(e.selfTime || 0).toFixed(2)}/${(e.duration || 0).toFixed(2)}`));
InspectorTest.completeTest();
}

Powered by Google App Engine
This is Rietveld 408576698