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

[web] Make our own icu_bidi for Skia #39200

Merged
merged 2 commits into from
Jan 30, 2023
Merged

Conversation

mdebbar
Copy link
Contributor

@mdebbar mdebbar commented Jan 27, 2023

The contents of the BUILD.gn file were reviewed by @bungeman (Ben Wagner) in this comment thread: https://skia-review.googlesource.com/c/skia/+/616799/comment/edb0aa3a_af1ca5bd/

Fixes flutter/flutter#119349

Copy link
Contributor

@eyebrowsoffire eyebrowsoffire left a comment

Choose a reason for hiding this comment

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

LGTM!

@mdebbar mdebbar added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 30, 2023
@auto-submit auto-submit bot merged commit 7cb1f0c into flutter:main Jan 30, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 31, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jan 31, 2023
…119586)

* e67a30e00 [web] @js('Intl') (flutter/engine#39211)

* 7cb1f0ce7 [web] Use our own icu_bidi (flutter/engine#39200)

* 772188181 @alwaysThrows is deprecated. Return `Never` instead. (flutter/engine#39269)
@mdebbar mdebbar deleted the icu_bidi_dir branch June 22, 2023 21:38
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[web] Make our own icu_bidi for Skia to consume
2 participants