Revert "Multiview pipeline Pt. 1: Skip illegal render calls (flutter#49266)"

This reverts commit 75f3a9c.

Author: dkwingsmt

shell/common/animator.cc

@@ -143,14 +143,16 @@ void Animator::BeginFrame(
 
 void Animator::Render(std::unique_ptr<flutter::LayerTree> layer_tree,
                       float device_pixel_ratio) {
-  // Animator::Render should be called after BeginFrame, which is indicated by
-  // frame_timings_recorder_ being non-null. Otherwise, this call is ignored.
-  if (frame_timings_recorder_ == nullptr) {
-    return;
-  }
-
   has_rendered_ = true;
 
+  if (!frame_timings_recorder_) {
+    // Framework can directly call render with a built scene.
+    frame_timings_recorder_ = std::make_unique<FrameTimingsRecorder>();
+    const fml::TimePoint placeholder_time = fml::TimePoint::Now();
+    frame_timings_recorder_->RecordVsync(placeholder_time, placeholder_time);
+    frame_timings_recorder_->RecordBuildStart(placeholder_time);
+  }
+
   TRACE_EVENT_WITH_FRAME_NUMBER(frame_timings_recorder_, "flutter",
                                 "Animator::Render", /*flow_id_count=*/0,
                                 /*flow_ids=*/nullptr);

shell/common/animator_unittests.cc

@@ -158,30 +158,20 @@ TEST_F(ShellTest, AnimatorDoesNotNotifyIdleBeforeRender) {
   latch.Wait();
   ASSERT_FALSE(delegate.notify_idle_called_);
 
-  fml::AutoResetWaitableEvent render_latch;
   // Validate it has not notified idle and try to render.
   task_runners.GetUITaskRunner()->PostDelayedTask(
       [&] {
         ASSERT_FALSE(delegate.notify_idle_called_);
-        EXPECT_CALL(delegate, OnAnimatorBeginFrame).WillOnce([&] {
-          auto layer_tree = std::make_unique<LayerTree>(
-              LayerTree::Config(), SkISize::Make(600, 800));
-          animator->Render(std::move(layer_tree), 1.0);
-          render_latch.Signal();
-        });
-        // Request a frame that builds a layer tree and renders a frame.
-        // When the frame is rendered, render_latch will be signaled.
-        animator->RequestFrame(true);
+        auto layer_tree = std::make_unique<LayerTree>(LayerTree::Config(),
+                                                      SkISize::Make(600, 800));
+        animator->Render(std::move(layer_tree), 1.0);
         task_runners.GetPlatformTaskRunner()->PostTask(flush_vsync_task);
       },
       // See kNotifyIdleTaskWaitTime in animator.cc.
       fml::TimeDelta::FromMilliseconds(60));
   latch.Wait();
-  render_latch.Wait();
 
-  // A frame has been rendered, and the next frame request will notify idle.
-  // But at the moment there isn't another frame request, therefore it still
-  // hasn't notified idle.
+  // Still hasn't notified idle because there has been no frame request.
   task_runners.GetUITaskRunner()->PostTask([&] {
     ASSERT_FALSE(delegate.notify_idle_called_);
     // False to avoid getting cals to BeginFrame that will request more frames
@@ -249,16 +239,16 @@ TEST_F(ShellTest, AnimatorDoesNotNotifyDelegateIfPipelineIsNotEmpty) {
 
   for (int i = 0; i < 2; i++) {
     task_runners.GetUITaskRunner()->PostTask([&] {
-      EXPECT_CALL(delegate, OnAnimatorBeginFrame).WillOnce([&] {
-        auto layer_tree = std::make_unique<LayerTree>(LayerTree::Config(),
-                                                      SkISize::Make(600, 800));
-        animator->Render(std::move(layer_tree), 1.0);
-        begin_frame_latch.Signal();
-      });
       animator->RequestFrame();
       task_runners.GetPlatformTaskRunner()->PostTask(flush_vsync_task);
     });
     begin_frame_latch.Wait();
+
+    PostTaskSync(task_runners.GetUITaskRunner(), [&] {
+      auto layer_tree = std::make_unique<LayerTree>(LayerTree::Config(),
+                                                    SkISize::Make(600, 800));
+      animator->Render(std::move(layer_tree), 1.0);
+    });
   }
 
   PostTaskSync(task_runners.GetUITaskRunner(), [&] { animator.reset(); });