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

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.

…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.
Copy link
Contributor

@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
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants