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

Simplify PlatformViewRenderTarget interface #49639

Merged

Conversation

johnmccutchan
Copy link
Contributor

  • Enable ImageReader/SurfaceProducer backends for Android >= 29.
  • This removes the need for a weird Android 29-specific fix in SurfaceTexturePlatformViewRenderTarget.
  • Now that we don't need the weird fix we can simplify the PlatformViewRenderTarget interface.

- Enable ImageReader/SurfaceProducer backends for Android >= 29.
- This removes the need for a weird fix in SurfaceTexturePlatformViewRenderTarget.
- Now that we don't need the weird fix we can simplify the PlatformViewRenderTarget interface.
@johnmccutchan johnmccutchan added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 9, 2024
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jan 9, 2024
Copy link
Contributor

auto-submit bot commented Jan 9, 2024

auto label is removed for flutter/engine/49639, due to - The status or check suite Mac mac_host_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.

Copy link
Contributor

@matanlurey matanlurey left a comment

Choose a reason for hiding this comment

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

Thanks! Simplification is always appreciated :)

@johnmccutchan johnmccutchan merged commit a35e3b0 into flutter:main Jan 9, 2024
24 of 26 checks passed
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
zijiehe-google-com pushed a commit to zijiehe-google-com/engine that referenced this pull request Jan 9, 2024
- Enable ImageReader/SurfaceProducer backends for Android >= 29.
- This removes the need for a weird Android 29-specific fix in
SurfaceTexturePlatformViewRenderTarget.
- Now that we don't need the weird fix we can simplify the
PlatformViewRenderTarget interface.
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.

2 participants