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

[Impeller] account for negative scale in max basis xy. #54630

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

jonahwilliams
Copy link
Member

Otherwise we get funky negative scales that break tessellation.

Copy link
Member

@bdero bdero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 19, 2024
@auto-submit auto-submit bot merged commit 390cb99 into flutter:main Aug 19, 2024
30 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 20, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 20, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Aug 20, 2024
…153739)

flutter/engine@5355c0b...86f269b

2024-08-20 [email protected] Roll Skia from e65e02c50d66 to 30a00629e135 (3 revisions) (flutter/engine#54634)
2024-08-20 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Changes DlColor to support wide gamut colors (#54473)" (flutter/engine#54636)
2024-08-19 [email protected] [Impeller] account for negative scale in max basis xy. (flutter/engine#54630)
2024-08-19 [email protected] Changes DlColor to support wide gamut colors (flutter/engine#54473)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
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: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Buchimi pushed a commit to Buchimi/flutter that referenced this pull request Sep 2, 2024
…lutter#153739)

flutter/engine@5355c0b...86f269b

2024-08-20 [email protected] Roll Skia from e65e02c50d66 to 30a00629e135 (3 revisions) (flutter/engine#54634)
2024-08-20 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Changes DlColor to support wide gamut colors (flutter#54473)" (flutter/engine#54636)
2024-08-19 [email protected] [Impeller] account for negative scale in max basis xy. (flutter/engine#54630)
2024-08-19 [email protected] Changes DlColor to support wide gamut colors (flutter/engine#54473)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
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: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App e: impeller
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants