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

[web] dont look up webgl params if no GPU is available #38948

Merged
merged 1 commit into from
Jan 19, 2023

Conversation

jonahwilliams
Copy link
Member

Potentially fixes flutter/flutter#118633

The glcontext being 0 indicates we are going to use software rendering, so don't try to look up gl parameters.

@flutter-dashboard flutter-dashboard bot added the platform-web Code specifically for the web engine label Jan 17, 2023
@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat (don't just cc him here, he won't see it! He's on Discord!).

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

Copy link
Contributor

@yjbanov yjbanov left a comment

Choose a reason for hiding this comment

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

If this is testable, would be good to have a test, but if we don't have the right test environment (e.g. it needs non-headless browser) then maybe this is the best we can do.

LGTM.

@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 19, 2023
@auto-submit auto-submit bot merged commit 78bbea0 into flutter:main Jan 19, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 19, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 19, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jan 19, 2023
…118829)

* a0e3c14d4 Merge MSAA alert functionality with UIA (flutter/engine#38745)

* 78bbea005 [web] dont look up webgl params if no GPU is available (flutter/engine#38948)
ricardoamador pushed a commit to ricardoamador/engine that referenced this pull request Jan 25, 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 needs tests platform-web Code specifically for the web engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pointer Interceptor faling integration test
2 participants