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

Conversation

@chinmaygarde
Copy link
Member

@chinmaygarde chinmaygarde added the e: libimpeller The standalone Impeller library with a single-header API. label Oct 4, 2024
@chinmaygarde chinmaygarde self-assigned this Oct 4, 2024
@flutter-dashboard

This comment was marked as outdated.

@chinmaygarde
Copy link
Member Author

This should work but I'm wiring up the test harness to work with "external" textures. Labeling this WIP till I sort out the test.

@lyceel, once this lands, all the deliverables we discussed earlier this week should be done!

@chinmaygarde chinmaygarde requested a review from zanderso October 4, 2024 20:23
@chinmaygarde chinmaygarde added the Work in progress (WIP) Not ready (yet) for review! label Oct 4, 2024
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

@chinmaygarde
Copy link
Member Author

Added a test.

@chinmaygarde chinmaygarde added autosubmit Merge PR when tree becomes green via auto submit App and removed Work in progress (WIP) Not ready (yet) for review! labels Oct 8, 2024
@auto-submit auto-submit bot merged commit 766b7c8 into flutter:main Oct 8, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 8, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 8, 2024
…156435)

flutter/engine@ea4a00f...0e7344a

2024-10-08 [email protected] [Impeller] libImpeller: Fix typo in public API. (flutter/engine#55750)
2024-10-08 [email protected] [Impeller] libImpeller: Allow wrapping external texture handles. (flutter/engine#55664)
2024-10-08 [email protected] Added mutex to the pending gpu tasks deque. (flutter/engine#55748)
2024-10-08 [email protected] Roll Skia from 857248fe0a9a to 8a2fe88d31e3 (7 revisions) (flutter/engine#55749)

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

Labels

autosubmit Merge PR when tree becomes green via auto submit App e: impeller e: libimpeller The standalone Impeller library with a single-header API.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Impeller] Impeller C API needs to be able to wrap external OpenGL textures.

2 participants