1400386 - Fix or disable test layout/reftests/font-face/font-display-1.html

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement, P1)

Description

[Michal Novotny [:michal]]

This test intermittently fails (bug 1238222) and there is a spike when we enable RCWN. For this reason RCWN was backed out several times (see bug 1392841 comments 19 and 33). This test needs to be either fixed or disabled.

[Tracking Requested - why for this release]:
It blocks RCWN which is targeted for 57 release.

Comment 1

[Brad Werth [:bradwerth]]

Attachment: Bug 1400386: Skip reftest layout/reftests/font-face/font-display-1.html for being intermittent.

Review commit: https://reviewboard.mozilla.org/r/180458/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/180458/

Comment 2

[Brad Werth [:bradwerth]]

I don't have insights into how this test fails or how to fix it. I defer to Jonathan Kew's analysis in Bug 1238222 comment 49. So I've provided a patch to disable the test.

Comment 3

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Comment on attachment 8908847 [details]
Bug 1400386: Skip reftest layout/reftests/font-face/font-display-1.html for being intermittent.

https://reviewboard.mozilla.org/r/180458/#review185706

::: layout/reftests/font-face/reftest.list:184
(Diff revision 1)
>  HTTP(..) == reflow-sanity-delay-1b.html reflow-sanity-1-ref.html
>  HTTP(..) == reflow-sanity-delay-1c.html reflow-sanity-1-ref.html
>  HTTP(..) == reflow-sanity-delay-1-metrics.html reflow-sanity-1-ref.html
>  
>  # font-display
> -pref(layout.css.font-display.enabled,true) HTTP(..) == font-display-1.html font-display-1-ref.html # normal font load (~500ms)
> +skip pref(layout.css.font-display.enabled,true) HTTP(..) == font-display-1.html font-display-1-ref.html # normal font load (~500ms)

All failures are on Linux and all except one on 32-bit: https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1238222
So keeping test coverage on Linux x64 should cause no issues. Please use
skip-if(/^Linux\x20i686/.test(http.oscpu))
similar to https://dxr.mozilla.org/mozilla-central/rev/f9a5e9ed62103c84e4cde915f4d08f1ce71be83e/image/test/reftest/downscaling/reftest.list#90

Comment 4

[Brad Werth [:bradwerth]]

Comment on attachment 8908847 [details]
Bug 1400386: Skip reftest layout/reftests/font-face/font-display-1.html for being intermittent.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/180458/diff/1-2/

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Comment on attachment 8908847 [details]
Bug 1400386: Skip reftest layout/reftests/font-face/font-display-1.html for being intermittent.

https://reviewboard.mozilla.org/r/180458/#review186080

Thank you for the change.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/203c1f909ae0
Skip reftest layout/reftests/font-face/font-display-1.html for being intermittent. r=aryx

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/203c1f909ae0