-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add a test for document.fonts.ready timing assumptions (#18489)
In #17219 (comment) a number of tests were found to depend on document.fonts.ready in this way, and it turns out it doesn't work in Safari with web fonts. Add a test to surface this problem more clearly.
- Loading branch information
Showing
1 changed file
with
32 additions
and
0 deletions.
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
<!DOCTYPE HTML> | ||
<title>document.fonts.ready resolves after layout depending on loaded fonts</title> | ||
<link rel="help" href="https://drafts.csswg.org/css-font-loading/#fontfaceset-pending-on-the-environment"> | ||
<script src="/resources/testharness.js"></script> | ||
<script src="/resources/testharnessreport.js"></script> | ||
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" /> | ||
<style> | ||
#foo { | ||
font: 100px/1 Ahem; | ||
} | ||
</style> | ||
<div id="log"></div> | ||
<span id="foo">X</span> | ||
<script> | ||
// The purpose of this test is to ensure that testharness.js tests can use | ||
// `document.fonts.ready` to wait for a web font to load, without having to | ||
// wait for the window load event before or requestAnimationFrame after. | ||
// | ||
// The spec says that a FontFaceSet is "pending on the environment" if "the | ||
// document has pending layout operations which might cause the user agent to | ||
// request a font, or which depend on recently-loaded fonts", and both are | ||
// assumed to hold true in this test. | ||
async_test(t => { | ||
assert_equals(document.fonts.size, 1, 'one font is pending'); | ||
document.fonts.ready.then(t.step_func_done(() => { | ||
const span = document.getElementById('foo'); | ||
const rect = span.getBoundingClientRect(); | ||
// If Ahem has loaded, the X will be 100px wide. | ||
assert_equals(rect.width, 100, 'span is 100px wide'); | ||
})); | ||
}); | ||
</script> |