Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: upgrade native sdk 6.2.6-sp.426142.b.2 #2002

Conversation

littleGnAl
Copy link
Contributor

Update native sdk 6.2.6-sp.426142.b.2 dependencies

This pull request is trigger by bot, you can checkout this branch and update it.

We need this
change(flutter/engine@90815e5),
which was released in Flutter SDK 3.7.0 to fix a crash on
Windows(#1999)

This PR is preparing for the fix.
We can't delete the `TextureRender` in the
`VideoViewController::DestroyTextureRender`, because the
`flutter::TextureRegistrar::UnregisterTexture` is called in the Flutter
rasterizer thread, it's not thread-safe.

We change the `VideoViewController::renderers_` to `std::map<int64_t,
TextureRender *>`, which holds the raw pointer `TextureRender *`, and
delete the `TextureRender ` in the
`flutter::TextureRegistrar::UnregisterTexture` callback which introduces
from Flutter SDK 3.7.0.
The `IrisRtcEngineRendering` handle becomes invalid after
`RtcEngine.release`, which needs explicit clean-up of the texture
renderers on the native side.

Fix #1567
@littleGnAl littleGnAl merged commit 73378b9 into special/native-6.2.6-sp.426142.b.2 Sep 13, 2024
40 of 47 checks passed
@littleGnAl littleGnAl deleted the littlegnal/special/native-6.2.6-sp.426142.b.2 branch September 13, 2024 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant