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

Revert "[Impeller] Provide the clear color to an advanced blend if it was optimized out" - #49064

Merged
auto-submit[bot] merged 1 commit into
flutter-team-archive:mainfrom
jason-simmons:revert_pr_48646
Dec 15, 2023
Merged

auto-submit[bot] merged 1 commit into
flutter-team-archive:mainfrom
jason-simmons:revert_pr_48646

Conversation

@jason-simmons

Copy link
Copy Markdown
Contributor

This reverts commit c32eb04.

That PR was outputting incorrect colors on the Play/AiksTest.BlendMode advanced blend tests.

… was optimized out"

This reverts commit c32eb04.

That PR was outputting incorrect colors on the Play/AiksTest.BlendMode
advanced blend tests.
@flutter-dashboard

Copy link
Copy Markdown

Golden file changes have been found for this pull request. Click here to view and triage (e.g. because this is an intentional change).

If you are still iterating on this change and are not ready to resolve the images on the Flutter Gold dashboard, consider marking this PR as a draft pull request above. You will still be able to view image results on the dashboard, commenting will be silenced, and the check will not try to resolve itself until marked ready for review.

Changes reported for pull request #49064 at sha 2668106

@jonahwilliams jonahwilliams left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jason-simmons jason-simmons added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 15, 2023
@auto-submit
auto-submit Bot merged commit db98451 into flutter-team-archive:main Dec 15, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 15, 2023
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Dec 15, 2023
flutter-team-archive/engine@b363740...db98451

2023-12-15 [email protected] Revert "[Impeller] Provide the clear color to an advanced blend if it was optimized out" (flutter-team-archive/engine#49064)

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],[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 will affect goldens

Development

Successfully merging this pull request may close these issues.

2 participants