Initial Timeline Events

runtime/vm/dart_api_impl.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "include/dart_api.h"
 #include "include/dart_mirrors_api.h"
 #include "include/dart_native_api.h"
 
 #include "platform/assert.h"
 #include "vm/class_finalizer.h"
@@ skipping 18 matching lines @@
 #include "vm/port.h"
 #include "vm/profiler.h"
 #include "vm/resolver.h"
 #include "vm/reusable_handles.h"
 #include "vm/service_event.h"
 #include "vm/service_isolate.h"
 #include "vm/service.h"
 #include "vm/stack_frame.h"
 #include "vm/symbols.h"
 #include "vm/tags.h"
+#include "vm/timeline.h"
 #include "vm/timer.h"
 #include "vm/unicode.h"
 #include "vm/verifier.h"
 #include "vm/version.h"
 
 namespace dart {
 
 DECLARE_FLAG(bool, load_deferred_eagerly);
 DECLARE_FLAG(bool, print_class_table);
 DECLARE_FLAG(bool, verify_handles);
@@ skipping 5556 matching lines @@
 }
 
 
 DART_EXPORT void Dart_RegisterRootServiceRequestCallback(
     const char* name,
     Dart_ServiceRequestCallback callback,
     void* user_data) {
   Service::RegisterRootEmbedderCallback(name, callback, user_data);
 }
 
+
+DART_EXPORT void Dart_TimelineDuration(const char* label,
+                                       int64_t start_micros,
+                                       int64_t end_micros) {
+  Isolate* isolate = Isolate::Current();
+  CHECK_ISOLATE(isolate);

[siva, 2015/06/12 19:50:34]

Normally we do a check of all arguments in the API call implementations  and
report errors (as they are coming from external code).

e.g:
- NULL check for label
- end_micros being > start_micros

[Cutch, 2015/06/12 23:18:56]

Done.

+  TimelineStream* stream = isolate->GetEmbedderStream();
+  ASSERT(stream != NULL);
+  TimelineEvent* event = stream->RecordEvent();
+  if (event != NULL) {
+    event->Duration(stream, label, start_micros, end_micros);
+  }
+}
+
+
+DART_EXPORT void Dart_TimelineInstant(const char* label) {
+  Isolate* isolate = Isolate::Current();
+  CHECK_ISOLATE(isolate);

[siva, 2015/06/12 19:50:34]

Ditto comment about checking label parameter.

[Cutch, 2015/06/12 23:18:56]

Done.

+  TimelineStream* stream = isolate->GetEmbedderStream();
+  ASSERT(stream != NULL);
+  TimelineEvent* event = stream->RecordEvent();
+  if (event != NULL) {
+    event->Instant(stream, label);
+  }
+}
+
+
+DART_EXPORT int64_t Dart_TimelineAsyncBegin(const char* label) {
+  Isolate* isolate = Isolate::Current();
+  CHECK_ISOLATE(isolate);

[siva, 2015/06/12 19:50:34]

Ditto.

[Cutch, 2015/06/12 23:18:56]

Done.

+  TimelineStream* stream = isolate->GetEmbedderStream();
+  ASSERT(stream != NULL);
+  TimelineEvent* event = stream->RecordEvent();
+  if (event != NULL) {
+    return event->AsyncBegin(stream, label);
+  }
+  return -1;
+}
+
+
+DART_EXPORT void Dart_TimelineAsyncInstant(const char* label,
+                                           int64_t async_id) {
+  if (async_id < 0) {
+    return;
+  }

[siva, 2015/06/12 19:50:34]

Also doesn't async_id have to be a value that you was handed out above, is there
a way to verify that?

[Cutch, 2015/06/12 23:18:56]

Not right now. I can start tracking open async ids internally but I would prefer
to avoid the overhead.

+  Isolate* isolate = Isolate::Current();
+  CHECK_ISOLATE(isolate);
+  TimelineStream* stream = isolate->GetEmbedderStream();
+  ASSERT(stream != NULL);
+  TimelineEvent* event = stream->RecordEvent();
+  if (event != NULL) {
+    event->AsyncInstant(stream, label, async_id);
+  }
+}
+
+
+DART_EXPORT void Dart_TimelineAsyncEnd(const char* label, int64_t async_id) {
+  if (async_id < 0) {
+    return;
+  }
+  Isolate* isolate = Isolate::Current();
+  CHECK_ISOLATE(isolate);
+  TimelineStream* stream = isolate->GetEmbedderStream();
+  ASSERT(stream != NULL);
+  TimelineEvent* event = stream->RecordEvent();
+  if (event != NULL) {
+    event->AsyncEnd(stream, label, async_id);
+  }
+}
+
 }  // namespace dart