Skip to content

Conversation

@bdero
Copy link
Member

@bdero bdero commented Mar 21, 2025

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
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.

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

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

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

@matanlurey
Copy link
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

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

@OBsIV
Copy link

OBsIV commented Jun 17, 2025

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
Copy link

Hellomik2002 commented Jun 17, 2025

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