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

Unified Diff: runtime/vm/dart_api_impl_test.cc

Issue 1294023009: Switch to a VM wide timeline recorder (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: 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
Index: runtime/vm/dart_api_impl_test.cc
diff --git a/runtime/vm/dart_api_impl_test.cc b/runtime/vm/dart_api_impl_test.cc
index 3d0b9b848eb3bb1829525c1308918aff0672b967..1e521feb312f3029f681ecc7733d97dd9f58e339 100644
--- a/runtime/vm/dart_api_impl_test.cc
+++ b/runtime/vm/dart_api_impl_test.cc
@@ -9238,9 +9238,10 @@ TEST_CASE(Timeline_Dart_TimelineDuration) {
// Add a duration event.
Dart_TimelineDuration("testDurationEvent", 0, 1);
// Check that it is in the output.
- TimelineEventRecorder* recorder = isolate->timeline_event_recorder();
+ TimelineEventRecorder* recorder = Timeline::recorder();
JSONStream js;
- recorder->PrintJSON(&js);
+ TimelineEventFilter filter;
+ recorder->PrintJSON(&js, &filter);
EXPECT_SUBSTRING("testDurationEvent", js.ToCString());
}
@@ -9253,9 +9254,10 @@ TEST_CASE(Timeline_Dart_TimelineInstant) {
stream->set_enabled(true);
Dart_TimelineInstant("testInstantEvent");
// Check that it is in the output.
- TimelineEventRecorder* recorder = isolate->timeline_event_recorder();
+ TimelineEventRecorder* recorder = Timeline::recorder();
JSONStream js;
- recorder->PrintJSON(&js);
+ TimelineEventFilter filter;
+ recorder->PrintJSON(&js, &filter);
EXPECT_SUBSTRING("testInstantEvent", js.ToCString());
}
@@ -9273,9 +9275,10 @@ TEST_CASE(Timeline_Dart_TimelineAsyncDisabled) {
// Call Dart_TimelineAsyncEnd with a negative async_id.
Dart_TimelineAsyncEnd("testAsyncEvent", async_id);
// Check that testAsync is not in the output.
- TimelineEventRecorder* recorder = isolate->timeline_event_recorder();
+ TimelineEventRecorder* recorder = Timeline::recorder();
JSONStream js;
- recorder->PrintJSON(&js);
+ TimelineEventFilter filter;
+ recorder->PrintJSON(&js, &filter);
EXPECT_NOTSUBSTRING("testAsyncEvent", js.ToCString());
}
@@ -9294,9 +9297,10 @@ TEST_CASE(Timeline_Dart_TimelineAsync) {
Dart_TimelineAsyncEnd("testAsyncEvent", async_id);
// Check that it is in the output.
- TimelineEventRecorder* recorder = isolate->timeline_event_recorder();
+ TimelineEventRecorder* recorder = Timeline::recorder();
JSONStream js;
- recorder->PrintJSON(&js);
+ TimelineEventFilter filter;
+ recorder->PrintJSON(&js, &filter);
EXPECT_SUBSTRING("testAsyncEvent", js.ToCString());
}
@@ -9370,7 +9374,7 @@ TEST_CASE(Timeline_Dart_TimelineGetTrace) {
// Heartbeat test.
EXPECT_SUBSTRING("\"cat\":\"Compiler\"", buffer);
EXPECT_SUBSTRING("\"name\":\"CompileFunction\"", buffer);
- EXPECT_SUBSTRING("\"function\":\"main\"", buffer);
+ EXPECT_SUBSTRING("\"function\":\"::_main\"", buffer);
// Free buffer allocated by AppendStreamConsumer
free(data.buffer);
« no previous file with comments | « runtime/vm/dart_api_impl.cc ('k') | runtime/vm/isolate.h » ('j') | runtime/vm/service.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698