-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
libtxt: cache fallback fonts found by a Minikin font collection #14482
Merged
jason-simmons
merged 1 commit into
flutter:master
from
jason-simmons:txt_cache_minikin_fallback
Dec 13, 2019
Merged
libtxt: cache fallback fonts found by a Minikin font collection #14482
jason-simmons
merged 1 commit into
flutter:master
from
jason-simmons:txt_cache_minikin_fallback
Dec 13, 2019
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@wangying3426 This adds a cache of fallback fonts to |
If a new fallback font is discovered during paragraph layout, the fallback font cache in txt::FontCollection will use that font in future layouts. However, that cache is not available if the new fallback font needs to be used for other characters within the current layout. This PR adds a cache to minikin::FontCollection and checks whether fonts in the cache can handle a character before calling the fallback font provider. See flutter#13257
jason-simmons
force-pushed
the
txt_cache_minikin_fallback
branch
from
December 13, 2019 20:55
d8e72b4
to
433ca7c
Compare
GaryQian
approved these changes
Dec 13, 2019
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.
Thanks! LGTM!
Good! |
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 17, 2019
iskakaushik
pushed a commit
to flutter/flutter
that referenced
this pull request
Dec 18, 2019
* c260057 Roll src/third_party/dart fe9f30c3896e..8ad8abfb7463 (19 commits) (flutter/engine#14480) * 47c1dde Roll src/third_party/skia d0840ecf5831..ea47b0e65802 (9 commits) (flutter/engine#14481) * 35adf37 [web] Implement Paragraph.getLineBoundary (flutter/engine#14479) * 00cbfd3 [web] Remove Paragraph._lines which isn't necessary anymore (flutter/engine#14485) * d6172fd libtxt: cache fallback fonts found by a Minikin font collection (flutter/engine#14482) * 1912478 adding firefox unit tests to font loading (flutter/engine#14487) * 6c71f89 Introduce an auto-deletable SkiaObject; make SkPaint a SkiaObject (flutter/engine#14486) * 2026b83 Roll src/third_party/dart 8ad8abfb7463..83eeab1e63d9 (13 commits) (flutter/engine#14488) * b43d3fe Roll src/third_party/skia ea47b0e65802..cd9ad409a90d (7 commits) (flutter/engine#14490) * 3c85b82 Roll fuchsia/sdk/core/linux-amd64 from QZVxo... to q0tup... (flutter/engine#14494) * 3e3eb9c Roll src/third_party/dart 83eeab1e63d9..6c99171686be (5 commits) (flutter/engine#14492) * cbe0e6c Roll src/third_party/skia cd9ad409a90d..cf0e3c63fd22 (1 commits) (flutter/engine#14495) * a653bd8 Roll fuchsia/sdk/core/mac-amd64 from 5t09i... to esDH2... (flutter/engine#14493) * 17d8882 Roll src/third_party/dart 6c99171686be..09a0b040dd9b (1 commits) (flutter/engine#14497) * be9c7c9 Roll src/third_party/skia cf0e3c63fd22..9e7199561fc8 (1 commits) (flutter/engine#14499) * c504b8f Roll src/third_party/skia 9e7199561fc8..ef363a9ce692 (1 commits) (flutter/engine#14500) * cf20f87 Roll src/third_party/dart 09a0b040dd9b..ae3973da4023 (1 commits) (flutter/engine#14501) * bb3166b Roll src/third_party/dart ae3973da4023..24b99838f433 (3 commits) (flutter/engine#14502) * 87c1110 Roll src/third_party/skia ef363a9ce692..187cd367d388 (1 commits) (flutter/engine#14503) * 45c6c3a Roll src/third_party/skia 187cd367d388..706eb5788e8b (1 commits) (flutter/engine#14505) * d292380 Fix comments of iOS status bar height constant (flutter/engine#14496) * 1ed7247 Roll src/third_party/dart 24b99838f433..2e1dd98e5e7f (1 commits) (flutter/engine#14507) * 8712461 Allow custom embedders to post low memory notifications. (flutter/engine#14506) * 1d1eae2 Roll fuchsia/sdk/core/linux-amd64 from q0tup... to O6ELR... (flutter/engine#14512) * 8b47886 Roll src/third_party/dart c5c469088697..44ba80053396 (1 commits) (flutter/engine#14510) * ca799fa Roll src/third_party/skia 706eb5788e8b..9b59953e77b9 (3 commits) (flutter/engine#14511) * 8456411 Roll src/third_party/dart 44ba80053396..2eba06d26f71 (32 commits) (flutter/engine#14515) * 46d76eb Roll src/third_party/skia 9b59953e77b9..d78a9b45b9e5 (23 commits) (flutter/engine#14516) * 5f9e558 Forward low memory notifications from the shell to DartVM. (flutter/engine#14517) * dc59758 Apply SmoothPointerDataDispatcher to Fuchsia (flutter/engine#14514) * 9556f2c Roll src/third_party/dart 2eba06d26f71..b1afe2d0ebc1 (6 commits) (flutter/engine#14523) * e829384 Roll fuchsia/sdk/core/linux-amd64 from O6ELR... to ZHhOi... (flutter/engine#14525) * 0fbdeb8 Roll src/third_party/dart b1afe2d0ebc1..2d332ee1d3f1 (1 commits) (flutter/engine#14526) * 500f9ce Roll src/third_party/skia 6153165d78f2..fdb2b7d53038 (2 commits) (flutter/engine#14528)
filmil
pushed a commit
to filmil/engine
that referenced
this pull request
Mar 13, 2020
…ter#14482) If a new fallback font is discovered during paragraph layout, the fallback font cache in txt::FontCollection will use that font in future layouts. However, that cache is not available if the new fallback font needs to be used for other characters within the current layout. This PR adds a cache to minikin::FontCollection and checks whether fonts in the cache can handle a character before calling the fallback font provider. See flutter#13257
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If a new fallback font is discovered during paragraph layout, the fallback
font cache in txt::FontCollection will use that font in future layouts.
However, that cache is not available if the new fallback font needs to be
used for other characters within the current layout. This PR adds a cache
to minikin::FontCollection and checks whether fonts in the cache can handle
a character before calling the fallback font provider.
See #13257