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

Revert "[Impeller] Use a device buffer for SkBitmap allocation, use Linear texture on Metal backend." #41533

Merged
merged 1 commit into from
Apr 26, 2023

Conversation

jonahwilliams
Copy link
Contributor

Reverts #41374

Breaks on Simulators!

…inear texture on Metal backend. (#41374)"

This reverts commit fddd5ad.
@jonahwilliams
Copy link
Contributor Author

Two changes are needed:

  1. device_buffer->AsTexture should do the right thing on simulator, by falling back performing a copy
    2.SupportsSharedDeviceBufferTextureMemory should be false so that we do an update if the buffer changes

@jonahwilliams jonahwilliams requested a review from bdero April 26, 2023 23:05
@jonahwilliams jonahwilliams added the revert Label used to revert changes in a closed and merged pull request. label Apr 26, 2023
@auto-submit auto-submit bot merged commit 464abc7 into main Apr 26, 2023
@auto-submit auto-submit bot deleted the revert-41374-upload_with_shared_texture branch April 26, 2023 23:05
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 26, 2023
Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

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

LGTM

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 27, 2023
jonahwilliams pushed a commit to flutter/flutter that referenced this pull request Apr 27, 2023
…125593)

flutter/engine@cf97541...d4ca524

2023-04-27 [email protected] Roll Skia from 20a1c61c5597 to
d315ab065af3 (5 revisions) (flutter/engine#41535)
2023-04-26 [email protected] Revert "[Impeller] Use a device
buffer for SkBitmap allocation, use Linear texture on Metal backend."
(flutter/engine#41533)
2023-04-26 [email protected] [codesign] Add pinned xcode version as
property to mac android aot engine (flutter/engine#41518)
2023-04-26 [email protected] [Impeller] Coerce opaque ColorSourceContents
to Source (flutter/engine#41525)
2023-04-26 [email protected] Roll Skia from 3fea88565a83 to
20a1c61c5597 (3 revisions) (flutter/engine#41530)
2023-04-26 [email protected] [Impeller] partial repaint for
Impeller/iOS. (flutter/engine#40959)
2023-04-26 [email protected] Updated todo with
github issue link (flutter/engine#41517)
2023-04-26 [email protected] Roll Clang from 20d06c833d83
to 5344d8e10bb7 (flutter/engine#41524)
2023-04-26 [email protected] [Impeller] Use a device buffer for
SkBitmap allocation, use Linear texture on Metal backend.
(flutter/engine#41374)
2023-04-26 [email protected] Manual clang roll to
5344d8e10bb7d8672d4bfae8adb010465470d51b (flutter/engine#41520)
2023-04-26 [email protected] Download and use the goma
client from cipd (flutter/engine#41488)

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
@dnfield
Copy link
Contributor

dnfield commented Apr 27, 2023

I think this is the same reason my previous attempt at something sort of like this had to be reverted lol

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
revert Label used to revert changes in a closed and merged pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants