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

Resize the RenderTarget before creating and passing its Surface to the VirtualDisplay #49619

Conversation

johnmccutchan
Copy link
Contributor

This change ensures that we first set the default buffer size on the underlying SurfaceTexture before creating the Surface for it.

This fixes a bug that only occurs on Android devices running 28 or older: see flutter/flutter#141068.

This also removes some unnecessary init code and cleans up tests.

@johnmccutchan johnmccutchan force-pushed the fix_virtual_display_android_28_earlier branch from 2464856 to 8c3c05d Compare January 9, 2024 14:57
…e VirtualDisplay

This change ensures that we first set the default buffer size on the underlying
SurfaceTexture before creating the Surface for it.

This fixes a bug that only occurs on Android devices running 28 or older:
see flutter/flutter#141068.

This also removes some unnecessary init code and cleans up tests.
@johnmccutchan johnmccutchan force-pushed the fix_virtual_display_android_28_earlier branch from 8c3c05d to 5c270b8 Compare January 9, 2024 16:36
Copy link
Member

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@johnmccutchan johnmccutchan merged commit ad6f192 into flutter:main Jan 9, 2024
18 of 25 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 9, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 9, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jan 9, 2024
…141191)

flutter/engine@036b39f...a35e3b0

2024-01-09 [email protected] Simplify PlatformViewRenderTarget interface (flutter/engine#49639)
2024-01-09 [email protected] Roll Skia from 05039d6ed8d8 to 8c8be76a62c0 (3 revisions) (flutter/engine#49638)
2024-01-09 [email protected] Skip flaky test on mac in http_disallow_http_connections_test.dart (flutter/engine#49635)
2024-01-09 [email protected] Resize the RenderTarget before creating and passing its Surface to the VirtualDisplay (flutter/engine#49619)
2024-01-09 [email protected] Migrate to Flutter mirrors for Google-owned Github repositories (flutter/engine#49203)
2024-01-09 [email protected] Roll Skia from ee263c34e03d to 05039d6ed8d8 (2 revisions) (flutter/engine#49636)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@Zhang-Zhi-Jie
Copy link

The problem still occurs when I use flutter3.16.9 and Android 9

@Zhang-Zhi-Jie
Copy link

I want to know how to solve it in flutter3.16.9,thanks

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.

4 participants