New issue
Advanced search Search tips

Issue 859898 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 24
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 828317



Sign in to add a comment

Add PostScript Name and full font name matching to FontService

Project Member Reported by drott@chromium.org, Jul 3

Issue description

Add PostScript Name and full font name matching to FontService so that on Linux we can answer requests for src: local() in @font-face declarations more precisely.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 23

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac

commit 89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac
Author: Dominik Röttsches <drott@chromium.org>
Date: Thu Aug 23 18:35:54 2018

Add PostScript and full font name matching methods to font service

Preparation for matching src: local() descriptors in @font-face
declarations from Blink.

Unit test added which tests matching against the set of test fonts we
are using on Linux.

Bug:  859898 
Change-Id: I4b28e2ef01edbc01e5a30c24480ef893bd32c9b2
Reviewed-on: https://chromium-review.googlesource.com/1124697
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Cr-Commit-Position: refs/heads/master@{#585559}
[modify] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/BUILD.gn
[modify] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/font_loader_test.cc
[modify] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/font_service_app.cc
[modify] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/font_service_app.h
[add] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/fontconfig_matching.cc
[add] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/fontconfig_matching.h
[modify] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/public/cpp/font_loader.cc
[modify] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/public/cpp/font_loader.h
[modify] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/public/cpp/font_service_thread.cc
[modify] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/public/cpp/font_service_thread.h
[modify] https://crrev.com/89b2ab6373bbd9864f0a117cfb8aabc98a7e3aac/components/services/font/public/interfaces/font_service.mojom

Status: Fixed (was: Started)

Sign in to add a comment