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

Commit 168cc21

Browse files
committed
fix comment
1 parent 4c4b019 commit 168cc21

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

lib/web_ui/lib/src/engine/text/font_collection.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -200,9 +200,9 @@ class FontManager {
200200
final html.FontFace fontFace = html.FontFace(family, list);
201201
return fontFace.load().then((_) {
202202
html.document.fonts.add(fontFace);
203-
// It is possible there were paragraph measurements for this new font
204-
// before it is loaded. They were measured using fallback font, so we
205-
// should clear the cache
203+
// There might be paragraph measurements for this new font before it is
204+
// loaded. They were measured using fallback font, so we should clear the
205+
// cache.
206206
TextMeasurementService.clearCache();
207207
}, onError: (dynamic exception) {
208208
// Failures here will throw an html.DomException which confusingly

lib/web_ui/test/text/font_loading_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import 'package:ui/src/engine.dart';
1212
Future<void> main() async {
1313
await ui.webOnlyInitializeTestDomRenderer();
1414
group('loadFontFromList', () {
15-
const String _testFontUrl = 'packages/ui/assets/Roboto-Regular.ttf';
15+
const String _testFontUrl = 'packages/ui/assets/ahem.ttf';
1616

1717
tearDown(() {
1818
html.document.fonts.clear();

0 commit comments

Comments
 (0)