Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "[Impeller] Reland: Add Impeller Metal support in the embedder API (#42411)" #42593

Merged
merged 1 commit into from Jun 6, 2023

Conversation

zanderso
Copy link
Member

@zanderso zanderso commented Jun 6, 2023

Reverts #42545

Not sure if this is needed, but getting a revert started for the regression noted here: #42545 (comment).

@jonahwilliams
Copy link
Member

#42594 (comment) might be easier to land

@jonahwilliams
Copy link
Member

or cherry pick that is, if we need to

@zanderso
Copy link
Member Author

zanderso commented Jun 6, 2023

The reland (#42545) missed the beta cut, so no need to CP anything.

@flutter-dashboard
Copy link

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 #42593 at sha cf90132

@zanderso zanderso merged commit 350a12b into main Jun 6, 2023
35 checks passed
@zanderso zanderso deleted the revert-42545-bdero/reland-embedder-macos-desktop branch June 6, 2023 17:57
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 6, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jun 6, 2023
…128350)

flutter/engine@b6d37f8...4571695

2023-06-06 jacksongardner@google.com Use proper sampling options in Skwasm's drawAtlas. (flutter/engine#42595)
2023-06-06 zanderso@users.noreply.github.com Revert "[Impeller] Reland: Add Impeller Metal support in the embedder API (#42411)" (flutter/engine#42593)
2023-06-06 zanderso@users.noreply.github.com Fix raster cache optional value check logic (flutter/engine#42592)
2023-06-06 zhangzhijian.123@bytedance.com [Impeller] Fix the rendering issue when the tile mode of gradient is set to 'decal' (flutter/engine#42591)
2023-06-06 skia-flutter-autoroll@skia.org Roll Skia from e8a71c76b88f to 4065e92bf577 (1 revision) (flutter/engine#42587)
2023-06-06 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from vAwdggqMrL1yoH_Zn... to lpbkSRJBMkPs0FM7_... (flutter/engine#42590)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from vAwdggqMrL1y to lpbkSRJBMkPs

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 jacksongardner@google.com,rmistry@google.com,zra@google.com 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants