layout/reftests/font-face/src-list-local-full.html fails due to Vera -> DejaVu alias

RESOLVED FIXED in Firefox 52

Status

()

Core
Layout: Text
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: karlt, Assigned: karlt)

Tracking

Trunk
mozilla52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

DejaVu fonts include aliases to cover the Vera fonts from which they are
derived.

https://github.com/dejavu-fonts/dejavu-fonts/blob/358190f40e90e1b633b7a93f6a4b0da33d2b8cdd/fontconfig/57-dejavu-sans.conf#L30

  <alias binding="same">
    <family>Bitstream Vera Sans</family>
    <accept>
      <family>DejaVu Sans</family>
    </accept>
  </alias>

When layout/reftests/font-face/src-list-local-full-ref.html loads on a system
without BitStream Vera Sans, but with DejaVu Sans, the reference renders
DejaVu for Vera, but the test falls back to FreeSans.
Comment hidden (mozreview-request)

Comment 2

a year ago
mozreview-review
Comment on attachment 8800101 [details]
bug 1309405 adjust font list order so that test provides same fallback as fontconfig alias from Vera to DejaVu

https://reviewboard.mozilla.org/r/85132/#review83704
Attachment #8800101 - Flags: review?(jfkthame) → review+

Comment 3

a year ago
Pushed by ktomlinson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1e5f6ab96cce
adjust font list order so that test provides same fallback as fontconfig alias from Vera to DejaVu r=jfkthame

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/1e5f6ab96cce
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox52: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.