commit c44d049eddc04d1fe7ea833fbd8c9ac0e1050b3b
author Chinmay Garde <chinmaygarde@gmail.com> Thu Nov 03 17:04:21 2016 -0700
committer GitHub <noreply@github.com> Thu Nov 03 17:04:21 2016 -0700
tree f7ec34a5227a8c9be4147eea7ef10c6623f6f9ce
parent 3164a9764590d8af54b10d066773b00b28e6b9b2

Fix PipelineProducer traces. Also add back the OnDisplayLink trace on iOS. (#3201)

shell/platform/darwin/ios/BUILD.gn

diff --git a/shell/platform/darwin/ios/BUILD.gn b/shell/platform/darwin/ios/BUILD.gn
index 3a7d33d..8b1f169 100644
--- a/shell/platform/darwin/ios/BUILD.gn
+++ b/shell/platform/darwin/ios/BUILD.gn
@@ -53,6 +53,7 @@
   deps = [
     "//base:base",
     "//dart/runtime:libdart",
+    "//flutter/glue",
     "//flutter/lib/ui",
     "//flutter/shell/common",
     "//flutter/shell/gpu",

shell/platform/darwin/ios/framework/Source/vsync_waiter_ios.mm

diff --git a/shell/platform/darwin/ios/framework/Source/vsync_waiter_ios.mm b/shell/platform/darwin/ios/framework/Source/vsync_waiter_ios.mm
index 3bd8e0b..6e0a360 100644
--- a/shell/platform/darwin/ios/framework/Source/vsync_waiter_ios.mm
+++ b/shell/platform/darwin/ios/framework/Source/vsync_waiter_ios.mm
@@ -11,6 +11,7 @@
 #include <mach/mach_time.h>
 
 #include "flutter/common/threads.h"
+#include "flutter/glue/trace_event.h"
 #include "lib/ftl/logging.h"
 
 @interface VSyncClient : NSObject
@@ -20,6 +21,7 @@
 @implementation VSyncClient {
   CADisplayLink* _displayLink;
   shell::VsyncWaiter::Callback _pendingCallback;
+  bool _traceCounter;
 }
 
 - (instancetype)init {
@@ -44,6 +46,8 @@
 }
 
 - (void)onDisplayLink:(CADisplayLink*)link {
+  _traceCounter = !_traceCounter;
+  TRACE_COUNTER1("flutter", "OnDisplayLink", _traceCounter);
   ftl::TimePoint frame_time = ftl::TimePoint::Now();
   _displayLink.paused = YES;
   auto callback = std::move(_pendingCallback);

synchronization/pipeline.h

diff --git a/synchronization/pipeline.h b/synchronization/pipeline.h
index 55e9fc5..25ab096 100644
--- a/synchronization/pipeline.h
+++ b/synchronization/pipeline.h
@@ -5,13 +5,13 @@
 #ifndef SYNCHRONIZATION_PIPELINE_H_
 #define SYNCHRONIZATION_PIPELINE_H_
 
+#include "flutter/glue/trace_event.h"
+#include "flutter/synchronization/pipeline.h"
+#include "flutter/synchronization/semaphore.h"
+#include "lib/ftl/functional/closure.h"
 #include "lib/ftl/macros.h"
 #include "lib/ftl/memory/ref_counted.h"
 #include "lib/ftl/synchronization/mutex.h"
-#include "lib/ftl/functional/closure.h"
-#include "flutter/synchronization/semaphore.h"
-#include "flutter/synchronization/pipeline.h"
-#include "flutter/glue/trace_event.h"
 
 #include <memory>
 #include <queue>
@@ -74,7 +74,6 @@
 
     ProducerContinuation(Continuation continuation, size_t trace_id)
         : continuation_(continuation), trace_id_(trace_id) {
-      TRACE_EVENT_ASYNC_BEGIN0("flutter", "PipelineItem", trace_id_);
       TRACE_EVENT_ASYNC_BEGIN0("flutter", "PipelineProduce", trace_id_);
     }
 
@@ -129,8 +128,6 @@
 
     empty_.Signal();
 
-    TRACE_EVENT_ASYNC_END0("flutter", "PipelineItem", trace_id);
-
     return items_count > 0 ? PipelineConsumeResult::MoreAvailable
                            : PipelineConsumeResult::Done;
   }