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

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

Issue 710113002: DevTools: remove TimelineModel.Events.RecordAdded event (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Addressed comment Created 6 years, 1 month 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: LayoutTests/inspector/tracing/timeline-model.html
diff --git a/LayoutTests/inspector/tracing/timeline-model.html b/LayoutTests/inspector/tracing/timeline-model.html
index 27e4eb68eba9e64f173b655c8873275cb747ad94..4bab3df4ca365fdce416ddfde13d2b8080b6fd5d 100644
--- a/LayoutTests/inspector/tracing/timeline-model.html
+++ b/LayoutTests/inspector/tracing/timeline-model.html
@@ -50,18 +50,12 @@ function test()
{"name": "Program", "ts": 2009999, "ph": "E", args: {}, "tid": mainThread, "pid": 100, "cat":"disabled-by-default.devtools.timeline"}
];
- var recordsFromEvent = [];
- function onRecordAdded(event)
- {
- recordsFromEvent.push(event.data);
- }
var tracingTimelineModel = InspectorTest.tracingTimelineModel();
- tracingTimelineModel.addEventListener(WebInspector.TimelineModel.Events.RecordAdded, onRecordAdded);
tracingTimelineModel.setEventsForTest(commonMetadata.concat(traceEvents));
var records = tracingTimelineModel.records();
- InspectorTest.assertEquals(records.length, recordsFromEvent.length);
+ var topLevelRecordsCount = 2;
+ InspectorTest.assertEquals(topLevelRecordsCount, records.length);
for (var i = 0; i < records.length; ++i) {
- InspectorTest.assertEquals(records[i], recordsFromEvent[i]);
InspectorTest.dumpTimelineModelRecord(records[i]);
}
InspectorTest.completeTest();

Powered by Google App Engine
This is Rietveld 408576698