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
Implement text rendering in Skwasm #41832
Conversation
Golden file changes have been found for this pull request. Click here to view and triage (e.g. because this is an intentional change). If you are still iterating on this change and are not ready to resolve the images on the Flutter Gold dashboard, consider marking this PR as a draft pull request above. You will still be able to view image results on the dashboard, commenting will be silenced, and the check will not try to resolve itself until marked ready for review. |
Co-authored-by: Mouad Debbar <mouad.debbar@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…ions) (#126445) Manual roll requested by zra@google.com flutter/engine@78f41a8...10ac36c 2023-05-09 jacksongardner@google.com Implement text rendering in Skwasm (flutter/engine#41832) 2023-05-09 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from leCRDVJ8szOS2LsPV... to lf8VcONMWlne4oa3H... (flutter/engine#41876) 2023-05-09 30870216+gaaclarke@users.noreply.github.com [Impeller] started cleaning up pools when devices are deleted (flutter/engine#41857) 2023-05-09 skia-flutter-autoroll@skia.org Roll Fuchsia Mac SDK from c4JvQUEOBHgtRdNPc... to JiOACcaGrDphuHIql... (flutter/engine#41874) 2023-05-09 godofredoc@google.com Move linux_license to engine v2. (flutter/engine#41863) 2023-05-09 godofredoc@google.com Replace Windows Unopt with the engine v2 version. (flutter/engine#41861) Also rolling transitive DEPS: fuchsia/sdk/core/linux-amd64 from leCRDVJ8szOS to lf8VcONMWlne fuchsia/sdk/core/mac-amd64 from c4JvQUEOBHgt to JiOACcaGrDph 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 aaclarke@google.com,rmistry@google.com,zra@google.com 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
…ions) (flutter#126445) Manual roll requested by zra@google.com flutter/engine@78f41a8...10ac36c 2023-05-09 jacksongardner@google.com Implement text rendering in Skwasm (flutter/engine#41832) 2023-05-09 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from leCRDVJ8szOS2LsPV... to lf8VcONMWlne4oa3H... (flutter/engine#41876) 2023-05-09 30870216+gaaclarke@users.noreply.github.com [Impeller] started cleaning up pools when devices are deleted (flutter/engine#41857) 2023-05-09 skia-flutter-autoroll@skia.org Roll Fuchsia Mac SDK from c4JvQUEOBHgtRdNPc... to JiOACcaGrDphuHIql... (flutter/engine#41874) 2023-05-09 godofredoc@google.com Move linux_license to engine v2. (flutter/engine#41863) 2023-05-09 godofredoc@google.com Replace Windows Unopt with the engine v2 version. (flutter/engine#41861) Also rolling transitive DEPS: fuchsia/sdk/core/linux-amd64 from leCRDVJ8szOS to lf8VcONMWlne fuchsia/sdk/core/mac-amd64 from c4JvQUEOBHgt to JiOACcaGrDph 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 aaclarke@google.com,rmistry@google.com,zra@google.com 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
This implements full text rendering with the Skwasm renderer.
The font fallback logic has been refactored to decouple the font fallback manager from any CanvasKit-specific types and functions.
A chunk of CanvasKit text rendering tests have been ported over to the renderer-agnostic
ui
tests.Fixes flutter/flutter#126339