Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Revert "[Impeller Scene] Change how property resolution works to fix Animation blending; add mutation log to nodes; enable backface culling; add vertex color contribution back to meshes" - #38843

Closed
itsjustkevin wants to merge 1 commit into
mainfrom
revert-38766-bdero/ui-animation-driving
Closed

Revert "[Impeller Scene] Change how property resolution works to fix Animation blending; add mutation log to nodes; enable backface culling; add vertex color contribution back to meshes"#38843
itsjustkevin wants to merge 1 commit into
mainfrom
revert-38766-bdero/ui-animation-driving

Conversation

@itsjustkevin

Copy link
Copy Markdown
Contributor

Reverts #38766 which broke internal tests at b/265394242

…Animation blending; add mutation log to nodes; enable backface culling; add vertex color contribution back to meshes (#38766)"

This reverts commit 2b024cb.
@zanderso

Copy link
Copy Markdown
Contributor

I think a fix already landed in g3 for this here cl/501123168. Has FRoB not picked it up?

@zanderso

Copy link
Copy Markdown
Contributor

Ah, actually never mind. That CL excluded ui/painting/scene, but these build failures are in impeller/scene.

@zanderso

Copy link
Copy Markdown
Contributor

Depending on timing, reverting this may have wider impacts, so waiting for @bdero to weigh in.

@zanderso

Copy link
Copy Markdown
Contributor

I filed flutter/flutter#118464 for the jazzy install failure in the iOS presubmit checks.

@chinmaygarde

Copy link
Copy Markdown
Contributor

This was resolved via a fix forward.

@chinmaygarde
chinmaygarde deleted the revert-38766-bdero/ui-animation-driving branch January 20, 2023 01:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants