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

Unified Diff: runtime/vm/timeline.cc

Issue 1295223002: Drop support for recording Dart objects in a timeline trace (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
« no previous file with comments | « runtime/vm/timeline.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/timeline.cc
diff --git a/runtime/vm/timeline.cc b/runtime/vm/timeline.cc
index a4988525cc51854231f1fc2e0a9b298d9cb073b9..833d1f8b6199403df773689eb24ca4412abd70b5 100644
--- a/runtime/vm/timeline.cc
+++ b/runtime/vm/timeline.cc
@@ -259,20 +259,6 @@ void TimelineStream::Init(const char* name, bool enabled) {
}
-TimelineEvent* TimelineStream::StartEvent(const Object& obj) {
- if (!enabled_ || (recorder_ == NULL)) {
- return NULL;
- }
- ASSERT(name_ != NULL);
- ASSERT(recorder_ != NULL);
- TimelineEvent* event = recorder_->StartEvent(obj);
- if (event != NULL) {
- event->StreamInit(this);
- }
- return event;
-}
-
-
TimelineEvent* TimelineStream::StartEvent() {
if (!enabled_ || (recorder_ == NULL)) {
return NULL;
@@ -493,31 +479,6 @@ intptr_t TimelineEventRingRecorder::FindOldestBlockIndex() const {
}
-void TimelineEventRingRecorder::VisitObjectPointers(
- ObjectPointerVisitor* visitor) {
- visitor->VisitPointer(reinterpret_cast<RawObject**>(&event_objects_));
-}
-
-
-TimelineEvent* TimelineEventRingRecorder::StartEvent(const Object& obj) {
- TimelineEvent* event = StartEvent();
- if (event == NULL) {
- return NULL;
- }
- // Grab the thread's timeline event block which contains |event|.
- Thread* thread = Thread::Current();
- TimelineEventBlock* thread_block = thread->timeline_block();
- ASSERT(thread_block != NULL);
- ASSERT(thread_block->length() > 0);
- const intptr_t block_index = thread_block->block_index();
- const intptr_t event_objects_index =
- block_index * TimelineEventBlock::kBlockSize + thread_block->length() - 1;
- const Array& event_objects = Array::Handle(event_objects_);
- event_objects.SetAt(event_objects_index, obj);
- return event;
-}
-
-
TimelineEvent* TimelineEventRingRecorder::StartEvent() {
return ThreadBlockStartEvent();
}
@@ -546,19 +507,6 @@ void TimelineEventStreamingRecorder::PrintJSON(JSONStream* js) {
}
-void TimelineEventStreamingRecorder::VisitObjectPointers(
- ObjectPointerVisitor* visitor) {
- // no-op.
-}
-
-
-TimelineEvent* TimelineEventStreamingRecorder::StartEvent(
- const Object& object) {
- // The streaming recorder does not track Dart objects.
- return StartEvent();
-}
-
-
TimelineEvent* TimelineEventStreamingRecorder::StartEvent() {
TimelineEvent* event = new TimelineEvent();
return event;
@@ -601,17 +549,6 @@ TimelineEventBlock* TimelineEventEndlessRecorder::GetHeadBlock() {
}
-void TimelineEventEndlessRecorder::VisitObjectPointers(
- ObjectPointerVisitor* visitor) {
- // no-op.
-}
-
-
-TimelineEvent* TimelineEventEndlessRecorder::StartEvent(const Object& object) {
- return StartEvent();
-}
-
-
TimelineEvent* TimelineEventEndlessRecorder::StartEvent() {
return ThreadBlockStartEvent();
}
« no previous file with comments | « runtime/vm/timeline.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698