Skip to content

Commit dffd060

Browse files
jihyerishmoz-wptsync-bot
authored andcommitted
Fix tiny pixels differ around 'P' on Win
This patch fixes test failures because of tiny pixels differ around 'P' by specifying ahem font Differential Revision: https://phabricator.services.mozilla.com/D195752 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1868364 gecko-commit: 19ac5fd6eb5db83cc867576e87173e24f2d5143b gecko-reviewers: cathiechen
1 parent 988e5e6 commit dffd060

13 files changed

+35
-16
lines changed

css/css-contain/content-visibility/content-visibility-075-ref.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<title>CSS Content Visibility: auto + scrollIntoView/fragment nav when size estimate is off (reference)</title>
55
<link rel="author" title="Vladimir Levin" href="mailto:vmpstr@chromium.org">
66
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
7-
7+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
88
<script src="/common/reftest-wait.js"></script>
99

1010
<style>
@@ -18,6 +18,7 @@
1818
#target {
1919
position: absolute;
2020
bottom: 0;
21+
font: 25px/1 Ahem;
2122
}
2223
</style>
2324

css/css-contain/content-visibility/content-visibility-075.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
77
<link rel="match" href="content-visibility-075-ref.html">
88
<meta name="assert" content="With content-visibility: auto, scrollIntoView targets the right element">
9-
9+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
1010
<script src="/common/reftest-wait.js"></script>
1111

1212
<style>
@@ -21,6 +21,7 @@
2121
#target {
2222
position: absolute;
2323
bottom: 0;
24+
font: 25px/1 Ahem;
2425
}
2526
.before_target {
2627
height: 40000px;

css/css-contain/content-visibility/content-visibility-076.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
77
<link rel="match" href="content-visibility-075-ref.html">
88
<meta name="assert" content="With content-visibility: auto, fragment navigation targets the right element">
9-
9+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
1010
<script src="/common/reftest-wait.js"></script>
1111

1212
<style>
@@ -21,6 +21,7 @@
2121
#target {
2222
position: absolute;
2323
bottom: 0;
24+
font: 25px/1 Ahem;
2425
}
2526
.before_target {
2627
height: 40000px;

css/css-contain/content-visibility/content-visibility-079-ref.html

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,11 @@
44
<title>CSS Content Visibility: auto in overflow hidden paints (reference)</title>
55
<link rel="author" title="Vladimir Levin" href="mailto:vmpstr@chromium.org">
66
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
7-
8-
<p>Test passes if you see the word “PASS” below.
7+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
8+
<style>
9+
div {
10+
font: 25px/1 Ahem;
11+
}
12+
</style>
13+
<p>Test passes if you see a black rectangle below.
914
<div>PASS</div>

css/css-contain/content-visibility/content-visibility-079.html

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,16 @@
55
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
66
<link rel="match" href="content-visibility-079-ref.html">
77
<meta name="assert" content="content-visibility auto element paints in an overflow hidden element that is not sized">
8-
8+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
99
<style>
10-
.auto { content-visibility: auto; }
10+
.auto {
11+
content-visibility: auto;
12+
font: 25px/1 Ahem;
13+
}
1114
.overflow { overflow: hidden; }
1215
</style>
1316

14-
<p>Test passes if you see the word “PASS” below.
17+
<p>Test passes if you see a black rectangle below.
1518
<div class=overflow>
1619
<div class=auto>PASS</div>
1720
</div>

css/css-contain/content-visibility/content-visibility-vs-scrollIntoView-001-ref.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<link rel="author" title="Cathie Chen" href="mailto:cathiechen@igalia.com">
66
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
77
<meta name="assert" content="Test if target scrollIntoView is visible when it is inside a nested content-visibility: auto">
8-
8+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
99
<script src="/common/reftest-wait.js"></script>
1010

1111
<style>
@@ -18,6 +18,7 @@
1818
#target {
1919
position: absolute;
2020
bottom: 0;
21+
font: 25px/1 Ahem;
2122
}
2223

2324
.before_target {

css/css-contain/content-visibility/content-visibility-vs-scrollIntoView-001.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<link rel="match" href="content-visibility-vs-scrollIntoView-001-ref.html">
88
<meta name="assert"
99
content="Test if target scrollIntoView is visible when it is inside a nested content-visibility: auto">
10-
10+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
1111
<script src="/common/reftest-wait.js"></script>
1212

1313
<style>
@@ -24,6 +24,7 @@
2424
#target {
2525
position: absolute;
2626
bottom: 0;
27+
font: 25px/1 Ahem;
2728
}
2829

2930
.before_target {

css/css-contain/content-visibility/content-visibility-with-popover-top-layer-004.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,14 @@
66
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
77
<link rel="match" href="spacer-with-popover-top-layer-ref.html">
88
<meta name="assert" content="top layer popovers render under c-v auto">
9-
9+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
1010
<script src="/common/reftest-wait.js"></script>
1111

1212
<style>
1313
.box { width: 100px; height: 100px; border: 1px solid black; }
1414
.auto { content-visibility: auto }
1515
.spacer { width: 10px; height: 3000px; background: lightblue; }
16+
#popover { font: 25px/1 Ahem }
1617
</style>
1718

1819
<div class=spacer></div>

css/css-contain/content-visibility/content-visibility-with-popover-top-layer-005.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,14 @@
66
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
77
<link rel="match" href="spacer-with-popover-top-layer-ref.html">
88
<meta name="assert" content="top layer popovers render under c-v auto">
9-
9+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
1010
<script src="/common/reftest-wait.js"></script>
1111

1212
<style>
1313
.box { width: 100px; height: 100px; border: 1px solid black; }
1414
.auto { content-visibility: auto }
1515
.spacer { width: 10px; height: 3000px; background: lightblue; }
16+
#popover { font: 25px/1 Ahem; }
1617
</style>
1718

1819
<div class=spacer></div>

css/css-contain/content-visibility/content-visibility-with-top-layer-004.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,14 @@
66
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
77
<link rel="match" href="spacer-with-top-layer-ref.html">
88
<meta name="assert" content="top layer dialogs render under c-v auto">
9-
9+
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
1010
<script src="/common/reftest-wait.js"></script>
1111

1212
<style>
1313
.box { width: 100px; height: 100px; border: 1px solid black; }
1414
.auto { content-visibility: auto }
1515
.spacer { width: 10px; height: 3000px; background: lightblue; }
16+
#dialog { font: 25px/1 Ahem; }
1617
</style>
1718

1819
<div class=spacer></div>

0 commit comments

Comments
 (0)