Revert "Start animator paused (flutter#29007)"

This reverts commit 666994f.

Author: yx-mike

shell/common/animator.cc

@@ -27,6 +27,7 @@ Animator::Animator(Delegate& delegate,
     : delegate_(delegate),
       task_runners_(std::move(task_runners)),
       waiter_(std::move(waiter)),
+      dart_frame_deadline_(0),
 #if SHELL_ENABLE_METAL
       layer_tree_pipeline_(std::make_shared<LayerTreePipeline>(2)),
 #else   // SHELL_ENABLE_METAL
@@ -40,6 +41,11 @@ Animator::Animator(Delegate& delegate,
               : 2)),
 #endif  // SHELL_ENABLE_METAL
       pending_frame_semaphore_(1),
+      paused_(false),
+      regenerate_layer_tree_(false),
+      frame_scheduled_(false),
+      notify_idle_task_id_(0),
+      dimension_change_pending_(false),
       weak_factory_(this) {
 }
 
@@ -86,10 +92,10 @@ const char* Animator::FrameParity() {
   return (frame_number % 2) ? "even" : "odd";
 }
 
-static fml::TimePoint FxlToDartOrEarlier(fml::TimePoint time) {
-  auto dart_now = fml::TimeDelta::FromMicroseconds(Dart_TimelineGetMicros());
+static int64_t FxlToDartOrEarlier(fml::TimePoint time) {
+  int64_t dart_now = Dart_TimelineGetMicros();
   fml::TimePoint fxl_now = fml::TimePoint::Now();
-  return fml::TimePoint::FromEpochDelta(time - fxl_now + dart_now);
+  return (time - fxl_now).ToMicroseconds() + dart_now;
 }
 
 void Animator::BeginFrame(
@@ -147,7 +153,7 @@ void Animator::BeginFrame(
     delegate_.OnAnimatorBeginFrame(frame_target_time, frame_number);
   }
 
-  if (!frame_scheduled_ && has_rendered_) {
+  if (!frame_scheduled_) {
     // Under certain workloads (such as our parent view resizing us, which is
     // communicated to us by repeat viewport metrics events), we won't
     // actually have a frame scheduled yet, despite the fact that we *will* be
@@ -177,7 +183,6 @@ void Animator::BeginFrame(
 }
 
 void Animator::Render(std::unique_ptr<flutter::LayerTree> layer_tree) {
-  has_rendered_ = true;
   if (dimension_change_pending_ &&
       layer_tree->frame_size() != last_layer_tree_size_) {
     dimension_change_pending_ = false;
@@ -270,10 +275,7 @@ void Animator::AwaitVSync() {
         }
       });
 
-  if (has_rendered_) {
-    delegate_.OnAnimatorNotifyIdle(
-        dart_frame_deadline_.ToEpochDelta().ToMicroseconds());
-  }
+  delegate_.OnAnimatorNotifyIdle(dart_frame_deadline_);
 }
 
 void Animator::ScheduleSecondaryVsyncCallback(uintptr_t id,

shell/common/animator.h

@@ -106,18 +106,17 @@ class Animator final {
 
   std::unique_ptr<FrameTimingsRecorder> frame_timings_recorder_;
   uint64_t frame_request_number_ = 1;
-  fml::TimePoint dart_frame_deadline_;
+  int64_t dart_frame_deadline_;
   std::shared_ptr<LayerTreePipeline> layer_tree_pipeline_;
   fml::Semaphore pending_frame_semaphore_;
   LayerTreePipeline::ProducerContinuation producer_continuation_;
-  bool paused_ = true;
-  bool regenerate_layer_tree_ = false;
-  bool frame_scheduled_ = false;
-  int notify_idle_task_id_ = 0;
-  bool dimension_change_pending_ = false;
+  bool paused_;
+  bool regenerate_layer_tree_;
+  bool frame_scheduled_;
+  int notify_idle_task_id_;
+  bool dimension_change_pending_;
   SkISize last_layer_tree_size_ = {0, 0};
   std::deque<uint64_t> trace_flow_ids_;
-  bool has_rendered_ = false;
 
   fml::WeakPtrFactory<Animator> weak_factory_;
 

shell/common/engine.cc

@@ -263,6 +263,7 @@ tonic::DartErrorHandleType Engine::GetUIIsolateLastError() {
 
 void Engine::OnOutputSurfaceCreated() {
   have_surface_ = true;
+  StartAnimatorIfPossible();
   ScheduleFrame();
 }
 
@@ -462,7 +463,6 @@ std::string Engine::DefaultRouteName() {
 }
 
 void Engine::ScheduleFrame(bool regenerate_layer_tree) {
-  StartAnimatorIfPossible();
   animator_->RequestFrame(regenerate_layer_tree);
 }