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

Revert "Replace deprecated and internal Skia EncodedImage calls with public versions" #41595

Merged
merged 1 commit into from
Apr 28, 2023

Conversation

zanderso
Copy link
Member

Reverts #41368

This is blocking rolls into the framework. Let's revert until we sort out whether there's a workaround for the change to the unpremultiplied alpha issue.

@zanderso zanderso requested a review from jason-simmons April 28, 2023 20:01
@zanderso
Copy link
Member Author

@kjlubick

Copy link
Member

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@zanderso zanderso added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 28, 2023
@zanderso zanderso merged commit 2a84ea5 into main Apr 28, 2023
@zanderso zanderso deleted the revert-41368-image-generator branch April 28, 2023 20:41
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 28, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Apr 28, 2023
…125714)

flutter/engine@98b6fab...2a84ea5

2023-04-28 [email protected] Revert "Replace deprecated and internal Skia EncodedImage calls with public versions" (flutter/engine#41595)
2023-04-28 [email protected] Roll buildroot to c96c9d4641714301fab450a5f3b0f3c42712e1e3 (flutter/engine#41589)
2023-04-28 [email protected] [platform_view] Only dispose view when it is removed from the composition order (flutter/engine#41521)
2023-04-28 [email protected] Determine lifecycle by looking at window focus also (flutter/engine#41094)
2023-04-28 [email protected] Increase timeout for clang-tidy on mac (flutter/engine#41588)
2023-04-28 [email protected] [Impeller] Always enable validation for goldens (flutter/engine#41574)
2023-04-28 [email protected] fuchsia: do not read version_history.json (flutter/engine#41585)
2023-04-28 [email protected] Roll Dart SDK from c7160d4ea0b7 to 8e9bf2bb9878 (5 revisions) (flutter/engine#41586)
2023-04-28 [email protected] Replace deprecated and internal Skia EncodedImage calls with public versions (flutter/engine#41368)
2023-04-28 [email protected] Roll Skia from 05d09f28250a to 9867fa253064 (18 revisions) (flutter/engine#41584)

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://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
@kjlubick
Copy link
Contributor

kjlubick commented May 1, 2023

What issues were there? Why did no unit tests fail?

kjlubick added a commit to kjlubick/engine that referenced this pull request May 1, 2023
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants