Skip to content

[Flutter GPU] WIP: Fix Android/GLES bugs. - #165630

Closed
bdero wants to merge 3 commits into
flutter:masterfrom
bdero:bdero/gles-fixes
Closed

bdero wants to merge 3 commits into
flutter:masterfrom
bdero:bdero/gles-fixes

Conversation

@bdero

@bdero bdero commented Mar 21, 2025 •

Copy link
Copy Markdown
Member

Resolves #164278.
Resolves bdero/flutter_scene#35.
Resolves bdero/flutter_scene#39.
Partly fixes: #160948.

And add an android_engine_test scenario.

@bdero
bdero requested a review from matanlurey as a code owner March 21, 2025 03:56
@github-actions github-actions Bot added engine flutter/engine related. See also e: labels. e: impeller Impeller rendering backend issues and features requests labels Mar 21, 2025
? std::move(cached_program)
: std::make_shared<UniqueHandleGLES>(UniqueHandleGLES::MakeUntracked(
reactor, HandleType::kProgram))));
: std::make_shared<UniqueHandleGLES>(reactor, HandleType::kProgram)));

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

WIP. This needs a new endpoint. Pushing in current state to test the end-to-end test on CI.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Which endpoint is needed ? Can I implement it? Can you give more information about it ?

@matanlurey
matanlurey removed their request for review April 2, 2025 15:30
@matanlurey
matanlurey marked this pull request as draft April 2, 2025 15:30
@flutter-dashboard

Copy link
Copy Markdown

This pull request has been changed to a draft. The currently pending flutter-gold status will not be able to resolve until a new commit is pushed or the change is marked ready for review again.

For more guidance, visit Writing a golden file test for package:flutter.

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

@wangcker

Copy link
Copy Markdown

sorry,When will this draft be merged? I'm in a hurry to use it.

@matanlurey

Copy link
Copy Markdown
Contributor

Closing for now as this PR is stale.

@matanlurey matanlurey closed this Apr 16, 2025
@Hellomik2002

This comment was marked as off-topic.

@Hellomik2002

Copy link
Copy Markdown

Can you give mor information about functionality of endpoint that needed to finish this pull request ? I am very excited to help

@OBsIV

OBsIV commented Jun 17, 2025

Copy link
Copy Markdown

We too are interested in this fix. We are investigating Flutter as an alternative to Qt and require Flutter GPU. It's impossible to make progress given how frequently this crash happens. It's also concerning to us how long this bug has been allowed to stick around given how serious it is. Is this a common thing in Flutter?

@Hellomik2002

Hellomik2002 commented Jun 17, 2025 •

Copy link
Copy Markdown

We too are interested in this fix. We are investigating Flutter as an alternative to Qt and require Flutter GPU. It's impossible to make progress given how frequently this crash happens. It's also concerning to us how long this bug has been allowed to stick around given how serious it is. Is this a common thing in Flutter?

I noticed that @jonahwilliams doing some pull requests, I don't know why @bdero didn't finish. However I really can not understand what endpoint needed to finish. I can do some engine stuff but without context it is very difficult . I think we will wait Mr Jonah W.

@flutter flutter locked as resolved and limited conversation to collaborators Jun 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

e: impeller Impeller rendering backend issues and features requests engine flutter/engine related. See also e: labels.

Projects

None yet

5 participants