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

Unified Diff: third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-model.html

Issue 2636543003: Timeline: get rid of some rarely used test helper methods (Closed)
Patch Set: Created 3 years, 11 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/timeline-misc/timeline-model.html
diff --git a/third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-model.html b/third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-model.html
index 3973c64f4e1fdc9a3b4d882653b547aed3e0a49b..d72855c4332a5f07de59a26b9fc1b955b12e9312 100644
--- a/third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-model.html
+++ b/third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-model.html
@@ -52,12 +52,11 @@ function test()
];
var tracingTimelineModel = InspectorTest.createTimelineModelWithEvents(commonMetadata.concat(traceEvents));
- var records = tracingTimelineModel.records();
- var topLevelRecordsCount = 2;
- InspectorTest.assertEquals(topLevelRecordsCount, records.length);
- for (var i = 0; i < records.length; ++i) {
- InspectorTest.dumpTimelineModelRecord(records[i]);
- }
+ InspectorTest.forAllEvents(tracingTimelineModel.mainThreadEvents(), (event, stack) => {
+ const prefix = Array(stack.length + 1).join("----") + (stack.length ? "> " : "");
+ const details = Timeline.TimelineUIUtils.buildDetailsTextForTraceEvent(event, null) || "";
+ InspectorTest.addResult(`${prefix}${event.name}: ${details}`);
+ });
InspectorTest.completeTest();
}

Powered by Google App Engine
This is Rietveld 408576698