Skip to content

[CP-stable][Windows] Use ANGLE blit extension on GLES 2.0#170924

Merged
auto-submit[bot] merged 4 commits into
flutter:flutter-3.32-candidate.0from
flutteractionsbot:cp-stable-2e0b9f26cc572c58aeea1f6e769c9aa921fdfc0c
Jun 24, 2025
Merged

[CP-stable][Windows] Use ANGLE blit extension on GLES 2.0#170924
auto-submit[bot] merged 4 commits into
flutter:flutter-3.32-candidate.0from
flutteractionsbot:cp-stable-2e0b9f26cc572c58aeea1f6e769c9aa921fdfc0c

Conversation

@flutteractionsbot

@flutteractionsbot flutteractionsbot commented Jun 20, 2025

Copy link
Copy Markdown
Contributor

This pull request is created by automatic cherry pick workflow

Issue Link:

#169178

Changelog Description:

Fix Flutter Windows on devices that only support OpenGL ES 2, like computers with Intel graphics cards.

Impact Description:

Flutter Windows 3.29.0 and higher crashed immediately on startup on computers using Intel HD Graphics graphics cards.

Workaround:

None, other than downgrading your Flutter SDK to 3.27.4 or lower.

Risk:

What is the risk level of this cherry-pick?

  • Low
  • Medium
  • High

Test Coverage:

Are you confident that your fix is well-tested by automated tests?

  • Yes
  • No

Validation Steps:

This bug does not reproduce on Flutter's infrastructure. The fix was manually tested on several Windows devices, including affected devices that use Intel HD Graphics graphics cards.

This updates Flutter Windows to fall back to
[`glBlitFramebufferANGLE`](https://github.com/google/angle/blob/main/extensions/ANGLE_framebuffer_blit.txt)
if `glBlitFramebuffer` is not available. This makes Flutter Windows work
on devices where ANGLE supports only GLES 2.0.

Fixes: flutter#169178

### Background

ANGLE only implements GLES 2.0 on some Windows 10+ machines
([1](flutter#169178 (comment)),
[2](flutter#169178 (comment))).

Flutter Windows 3.27.4 and lower was using `glBlitFramebuffer` on GLES
2.0 devices, even though that requires GLES 3.0. This magically worked
though, thanks to ANGLE.

However in 3.29.0, Impeller was updated to not resolve GLES 3.0 procs on
GLES 2.0: flutter-team-archive/engine#56636. This caused
Flutter Windows to crash on GLES 2.0 devices since it was calling
`glBlitFramebuffer`.

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [x] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [x] I followed the [breaking change policy] and added [Data Driven
Fixes] where supported.
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview
[Tree Hygiene]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
[test-exempt]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
[Discord]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
[Data Driven Fixes]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
@flutteractionsbot flutteractionsbot added the cp: review Cherry-picks in the review queue label Jun 20, 2025
@flutteractionsbot

Copy link
Copy Markdown
Contributor Author

@loic-sharma please fill out the PR description above, afterwards the release team will review this request.

@flutter-dashboard

Copy link
Copy Markdown

This pull request was opened from and to a release candidate branch. This should only be done as part of the official Flutter release process. If you are attempting to make a regular contribution to the Flutter project, please close this PR and follow the instructions at Tree Hygiene for detailed instructions on contributing to Flutter.

Reviewers: Use caution before merging pull requests to release branches. Ensure the proper procedure has been followed.

@github-actions github-actions Bot added engine flutter/engine related. See also e: labels. platform-windows Building on or for Windows specifically a: desktop Running on desktop e: impeller Impeller rendering backend issues and features requests labels Jun 20, 2025
Comment thread CHANGELOG.md

@cbracken cbracken left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM stamp from a Japanese personal seal

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Jun 24, 2025
@auto-submit auto-submit Bot merged commit dd93de6 into flutter:flutter-3.32-candidate.0 Jun 24, 2025
165 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 25, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: desktop Running on desktop autosubmit Merge PR when tree becomes green via auto submit App cp: review Cherry-picks in the review queue e: impeller Impeller rendering backend issues and features requests engine flutter/engine related. See also e: labels. platform-windows Building on or for Windows specifically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants