Roll Dart SDK from 418bee5da2e2 to 8d7a6aabd3a3 (2 revisions) (flutter#38738)

https://dart.googlesource.com/sdk.git/+log/418bee5da2e2..8d7a6aabd3a3

2023-01-10 [email protected] Version 3.0.0-105.0.dev
2023-01-10 [email protected] Version 3.0.0-104.0.dev

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dart-sdk-flutter-engine
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter Engine: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Author: skia-flutter-autoroll

DEPS

@@ -48,7 +48,7 @@ vars = {
   # Dart is: https://github.com/dart-lang/sdk/blob/main/DEPS
   # You can use //tools/dart/create_updated_flutter_deps.py to produce
   # updated revision list of existing dependencies.
-  'dart_revision': '418bee5da2e2505a13d992a35389d6406cf1bfec',
+  'dart_revision': '8d7a6aabd3a381fe630c111746789ccb34b2c42a',
 
   # WARNING: DO NOT EDIT MANUALLY
   # The lines between blank lines above and below are generated by a script. See create_updated_flutter_deps.py
@@ -63,7 +63,7 @@ vars = {
   'dart_pub_rev': '0cbaf7a2fb8c8ca543c6f222bf25d5f5c63abbf3',
   'dart_root_certificates_rev': '692f6d6488af68e0121317a9c2c9eb393eb0ee50',
   'dart_watcher_rev': '2e0db7175016382c38e786356f6c36c6a2bd416a',
-  'dart_webdev_rev': '8795ca5b89df0f912e31b0f5fa372b90f2a0bf66',
+  'dart_webdev_rev': '49f97b8848c264e634affb045cf95ff4a34a7784',
   'dart_webkit_inspection_protocol_rev': 'ddb624cd85954dd384056cc253a8fc2b9da5364d',
   'dart_yaml_edit_rev': '299f74594ff9fda412c1da5c0b5d5231d0c6fc42',
   'dart_zlib_rev': '27c2f474b71d0d20764f86f60ef8b00da1a16cda',
@@ -340,7 +340,7 @@ deps = {
    Var('dart_git') + '/dart_style.git@d0c6b1da52c9f80679a7aaa05e5935ef5c6bb52f',
 
   'src/third_party/dart/third_party/pkg/dartdoc':
-   Var('dart_git') + '/dartdoc.git@9ed196f1cec6c764c22acaa6599de5c30e361d4a',
+   Var('dart_git') + '/dartdoc.git@29a1bbf8576f39c6f7c7dcf71f131a59bb346625',
 
   'src/third_party/dart/third_party/pkg/ffi':
    Var('dart_git') + '/ffi.git@2a56c2a9a87122c057168874bde384debbb806b0',
@@ -373,7 +373,7 @@ deps = {
    Var('dart_git') + '/json_rpc_2.git@bd9f8d90ec9ce9d0da5c11f3e71b87af3aaca3b4',
 
   'src/third_party/dart/third_party/pkg/linter':
-   Var('dart_git') + '/linter.git@79b843349deaea4a2455cfa37cc47cbc0e472190',
+   Var('dart_git') + '/linter.git@8f7b75f8a3501ca2c4d5bbc1f003d90d66447544',
 
   'src/third_party/dart/third_party/pkg/logging':
    Var('dart_git') + '/logging.git@b525d5c39c8ae6f0826c63a371d286080e2ec93e',
@@ -430,7 +430,7 @@ deps = {
    Var('dart_git') + '/stream_channel.git@0a7800ae41be5f7ba9990388fecc30a700186a48',
 
   'src/third_party/dart/third_party/pkg/string_scanner':
-   Var('dart_git') + '/string_scanner.git@6ddab2c6a3374c9b0fe5ba08ff9badce944a797e',
+   Var('dart_git') + '/string_scanner.git@c58618df0f17cbed664b3868c949abb13491e4d3',
 
   'src/third_party/dart/third_party/pkg/term_glyph':
    Var('dart_git') + '/term_glyph.git@2bf45943466a5bb2d8d1add714fbfeebe5cabab5',

ci/licenses_golden/licenses_third_party

@@ -1,4 +1,4 @@
-Signature: e597cfe931e7b4964f4d51218347aff0
+Signature: e0d2a30edffb57a66287705548386e33
 
 ====================================================================================================
 LIBRARY: angle
@@ -40027,6 +40027,7 @@ ORIGIN: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/class_id.dart + ../
 ORIGIN: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/convert_patch.dart + ../../../third_party/dart/LICENSE
 ORIGIN: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/core_patch.dart + ../../../third_party/dart/LICENSE
 ORIGIN: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/date_patch.dart + ../../../third_party/dart/LICENSE
+ORIGIN: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/deferred.dart + ../../../third_party/dart/LICENSE
 ORIGIN: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/developer.dart + ../../../third_party/dart/LICENSE
 ORIGIN: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/double.dart + ../../../third_party/dart/LICENSE
 ORIGIN: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/errors_patch.dart + ../../../third_party/dart/LICENSE
@@ -40113,6 +40114,7 @@ FILE: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/class_id.dart
 FILE: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/convert_patch.dart
 FILE: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/core_patch.dart
 FILE: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/date_patch.dart
+FILE: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/deferred.dart
 FILE: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/developer.dart
 FILE: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/double.dart
 FILE: ../../../third_party/dart/sdk/lib/_internal/wasm/lib/errors_patch.dart

impeller/scene/animation/animation_player.cc

@@ -19,10 +19,15 @@ AnimationPlayer::~AnimationPlayer() = default;
 AnimationPlayer::AnimationPlayer(AnimationPlayer&&) = default;
 AnimationPlayer& AnimationPlayer::operator=(AnimationPlayer&&) = default;
 
-AnimationClip& AnimationPlayer::AddAnimation(
-    std::shared_ptr<Animation> animation,
+AnimationClip* AnimationPlayer::AddAnimation(
+    const std::shared_ptr<Animation>& animation,
     Node* bind_target) {
-  AnimationClip clip(std::move(animation), bind_target);
+  if (!animation) {
+    VALIDATION_LOG << "Cannot add null animation.";
+    return nullptr;
+  }
+
+  AnimationClip clip(animation, bind_target);
 
   // Record all of the unique default transforms that this AnimationClip
   // will mutate.
@@ -31,8 +36,18 @@ AnimationClip& AnimationPlayer::AddAnimation(
         {binding.node, binding.node->GetLocalTransform()});
   }
 
-  clips_.push_back(std::move(clip));
-  return clips_.back();
+  clips_.insert({animation->GetName(), std::move(clip)});
+  auto found = clips_.find(animation->GetName());
+  FML_DCHECK(found != clips_.end());
+  return &found->second;
+}
+
+AnimationClip* AnimationPlayer::GetClip(const std::string& name) {
+  auto result = clips_.find(name);
+  if (result == clips_.end()) {
+    return nullptr;
+  }
+  return &result->second;
 }
 
 void AnimationPlayer::Update() {
@@ -46,7 +61,7 @@ void AnimationPlayer::Update() {
   Reset();
 
   // Update and apply all clips.
-  for (auto& clip : clips_) {
+  for (auto& [_, clip] : clips_) {
     clip.Advance(delta_time);
     clip.ApplyToBindings();
   }

impeller/scene/animation/animation_player.h

@@ -4,9 +4,9 @@
 
 #pragma once
 
+#include <map>
 #include <memory>
 #include <optional>
-#include <unordered_map>
 #include <vector>
 
 #include "flutter/fml/hash_combine.h"
@@ -29,9 +29,11 @@ class AnimationPlayer final {
   AnimationPlayer(AnimationPlayer&&);
   AnimationPlayer& operator=(AnimationPlayer&&);
 
-  AnimationClip& AddAnimation(std::shared_ptr<Animation> animation,
+  AnimationClip* AddAnimation(const std::shared_ptr<Animation>& animation,
                               Node* bind_target);
 
+  AnimationClip* GetClip(const std::string& name);
+
   /// @brief  Advanced all clips and updates animated properties in the scene.
   void Update();
 
@@ -41,7 +43,7 @@ class AnimationPlayer final {
  private:
   std::unordered_map<Node*, Matrix> default_target_transforms_;
 
-  std::vector<AnimationClip> clips_;
+  std::map<std::string, AnimationClip> clips_;
 
   std::optional<TimePoint> previous_time_;
 

impeller/scene/geometry.cc

@@ -95,7 +95,7 @@ std::shared_ptr<Geometry> Geometry::MakeFromFlatbuffer(
   }
 
   DeviceBufferDescriptor buffer_desc;
-  buffer_desc.size = vertices_bytes * indices_bytes;
+  buffer_desc.size = vertices_bytes + indices_bytes;
   buffer_desc.storage_mode = StorageMode::kHostVisible;
 
   auto buffer = allocator.CreateBuffer(buffer_desc);

impeller/scene/node.cc

@@ -7,6 +7,7 @@
 #include <inttypes.h>
 #include <atomic>
 #include <memory>
+#include <vector>
 
 #include "flutter/fml/logging.h"
 #include "impeller/base/strings.h"
@@ -24,6 +25,24 @@ namespace scene {
 
 static std::atomic_uint64_t kNextNodeID = 0;
 
+void Node::MutationLog::Append(const Entry& entry) {
+  std::lock_guard<std::mutex> lock(write_mutex_);
+  dirty_ = true;
+  entries_.push_back(entry);
+}
+
+std::optional<std::vector<Node::MutationLog::Entry>>
+Node::MutationLog::Flush() {
+  if (!dirty_) {
+    return std::nullopt;
+  }
+  std::lock_guard<std::mutex> lock(write_mutex_);
+  dirty_ = false;
+  auto result = entries_;
+  entries_ = {};
+  return result;
+}
+
 std::shared_ptr<Node> Node::MakeFromFlatbuffer(
     const fml::Mapping& ipscene_mapping,
     Allocator& allocator) {
@@ -257,7 +276,7 @@ std::shared_ptr<Animation> Node::FindAnimationByName(
   return nullptr;
 }
 
-AnimationClip& Node::AddAnimation(const std::shared_ptr<Animation>& animation) {
+AnimationClip* Node::AddAnimation(const std::shared_ptr<Animation>& animation) {
   if (!animation_player_.has_value()) {
     animation_player_ = AnimationPlayer();
   }
@@ -287,6 +306,11 @@ Matrix Node::GetGlobalTransform() const {
 }
 
 bool Node::AddChild(std::shared_ptr<Node> node) {
+  if (!node) {
+    VALIDATION_LOG << "Cannot add null child to node.";
+    return false;
+  }
+
   // TODO(bdero): Figure out a better paradigm/rules for nodes with multiple
   //              parents. We should probably disallow this, make deep
   //              copying of nodes cheap and easy, add mesh instancing, etc.
@@ -328,7 +352,32 @@ bool Node::IsJoint() const {
 
 bool Node::Render(SceneEncoder& encoder,
                   Allocator& allocator,
-                  const Matrix& parent_transform) const {
+                  const Matrix& parent_transform) {
+  std::optional<std::vector<MutationLog::Entry>> log = mutation_log_.Flush();
+  if (log.has_value()) {
+    for (const auto& entry : log.value()) {
+      if (auto e = std::get_if<MutationLog::SetTransformEntry>(&entry)) {
+        local_transform_ = e->transform;
+      } else if (auto e =
+                     std::get_if<MutationLog::SetAnimationStateEntry>(&entry)) {
+        auto* clip = animation_player_->GetClip(e->animation_name);
+        if (!clip) {
+          continue;
+        }
+        clip->SetPlaying(e->playing);
+        clip->SetWeight(e->weight);
+        clip->SetPlaybackTimeScale(e->time_scale);
+      } else if (auto e =
+                     std::get_if<MutationLog::SeekAnimationEntry>(&entry)) {
+        auto* clip = animation_player_->GetClip(e->animation_name);
+        if (!clip) {
+          continue;
+        }
+        clip->Seek(SecondsF(e->time));
+      }
+    }
+  }
+
   if (animation_player_.has_value()) {
     animation_player_->Update();
   }
@@ -345,5 +394,9 @@ bool Node::Render(SceneEncoder& encoder,
   return true;
 }
 
+void Node::AddMutation(const MutationLog::Entry& entry) {
+  mutation_log_.Append(entry);
+}
+
 }  // namespace scene
 }  // namespace impeller

impeller/scene/node.h

@@ -5,6 +5,7 @@
 #pragma once
 
 #include <memory>
+#include <mutex>
 #include <optional>
 #include <vector>
 
@@ -25,6 +26,39 @@ namespace scene {
 
 class Node final {
  public:
+  class MutationLog {
+   public:
+    struct SetTransformEntry {
+      Matrix transform;
+    };
+
+    struct SetAnimationStateEntry {
+      std::string animation_name;
+      bool playing;
+      float weight;
+      float time_scale;
+    };
+
+    struct SeekAnimationEntry {
+      std::string animation_name;
+      float time;
+    };
+
+    using Entry = std::
+        variant<SetTransformEntry, SetAnimationStateEntry, SeekAnimationEntry>;
+
+    void Append(const Entry& entry);
+
+   private:
+    std::optional<std::vector<Entry>> Flush();
+
+    bool dirty_ = false;
+    std::vector<Entry> entries_;
+    std::mutex write_mutex_;
+
+    friend Node;
+  };
+
   static std::shared_ptr<Node> MakeFromFlatbuffer(
       const fml::Mapping& ipscene_mapping,
       Allocator& allocator);
@@ -44,7 +78,7 @@ class Node final {
       bool exclude_animation_players = false) const;
 
   std::shared_ptr<Animation> FindAnimationByName(const std::string& name) const;
-  AnimationClip& AddAnimation(const std::shared_ptr<Animation>& animation);
+  AnimationClip* AddAnimation(const std::shared_ptr<Animation>& animation);
 
   void SetLocalTransform(Matrix transform);
   Matrix GetLocalTransform() const;
@@ -63,7 +97,9 @@ class Node final {
 
   bool Render(SceneEncoder& encoder,
               Allocator& allocator,
-              const Matrix& parent_transform) const;
+              const Matrix& parent_transform);
+
+  void AddMutation(const MutationLog::Entry& entry);
 
  private:
   void UnpackFromFlatbuffer(
@@ -72,6 +108,8 @@ class Node final {
       const std::vector<std::shared_ptr<Texture>>& textures,
       Allocator& allocator);
 
+  mutable MutationLog mutation_log_;
+
   Matrix local_transform_;
 
   std::string name_;

impeller/scene/scene.cc

@@ -31,7 +31,7 @@ Node& Scene::GetRoot() {
 }
 
 bool Scene::Render(const RenderTarget& render_target,
-                   const Matrix& camera_transform) const {
+                   const Matrix& camera_transform) {
   // Collect the render commands from the scene.
   SceneEncoder encoder;
   if (!root_.Render(encoder,
@@ -57,8 +57,7 @@ bool Scene::Render(const RenderTarget& render_target,
   return true;
 }
 
-bool Scene::Render(const RenderTarget& render_target,
-                   const Camera& camera) const {
+bool Scene::Render(const RenderTarget& render_target, const Camera& camera) {
   return Render(render_target,
                 camera.GetTransform(render_target.GetRenderTargetSize()));
 }

impeller/scene/scene.h

@@ -28,9 +28,9 @@ class Scene {
   Node& GetRoot();
 
   bool Render(const RenderTarget& render_target,
-              const Matrix& camera_transform) const;
+              const Matrix& camera_transform);
 
-  bool Render(const RenderTarget& render_target, const Camera& camera) const;
+  bool Render(const RenderTarget& render_target, const Camera& camera);
 
  private:
   std::shared_ptr<SceneContext> scene_context_;