This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Reverts "Remove #if SHELL_ENABLE_METAL checks in iOS code (#51636)" #51941
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reverts: #51636
Initiated by: jmagman
Reason for reverting: This caused a slew of unexpected Scuba changes b/333039358
Original PR Author: jmagman
Reviewed By: {cbracken}
This change reverts the following previous change:
All physical iOS devices Flutter supports (iOS 12+) can run Metal. The only time Flutter doesn't use Metal afaik is for iOS simulators running < iOS 13, which is covered by a few
if (@available(iOS METAL_IOS_VERSION_BASELINE, *))
checks.engine/shell/platform/darwin/ios/rendering_api_selection.h
Lines 37 to 41 in 8a51e12
Remove hardware checks for physical devices.
Remove
shell_enable_metal
from the gn files and the#if SHELL_ENABLE_METAL
checks in the iOS embedder.I limited this PR to just iOS, but I imagine it's safe to remove
shell_enable_metal
everywhere?engine/shell/platform/darwin/macos/BUILD.gn
Line 18 in 41da00a
engine/tools/gn
Lines 673 to 679 in 41da00a