New issue
Advanced search Search tips

Issue 867877 link

Starred by 1 user
Status: Fixed
Owner:
drott@chromium.org
Closed: Aug 1
Cc:
behdad@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 828317



Sign in to add a comment

Add font scanner to extract full font names and postscript names

Project Member Reported by drott@chromium.org, Jul 26 
On Android we do not have API for searching through the installed fonts by full font name or postscript name. Build our own scanner for the purpose.

This scanner is prepared to pass a readonly shared memory region to Blink, through which Blink can do font matching for the case of @font-face { src: local() }.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 1 

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

commit 7e00fed99a45512b17c974482c73ddcb72181850
Author: Dominik Röttsches <drott@chromium.org>
Date: Wed Aug 01 13:05:51 2018

Add unique font name lookup table implementation and tests

Preparation for hooking up @font-face { src: local() } matching in
Blink. For now adding an implementation that extracts full font name and
postscript name from fonts given a set of directories to scan.

Includes unit tests for basic functionality, compatibility with TrueType
collections and for resilience against corrupted font files.

For the general design see https://crbug.com/828317.

Bug:  867877 
Change-Id: I5f557d99a5be49671bd9aa13c1712b6f8a2aa96b
Reviewed-on: https://chromium-review.googlesource.com/1151348
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: David Turner <digit@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579785}
[modify] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/BUILD.gn
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/BUILD.gn
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/OWNERS
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/font_table_matcher.cc
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/font_table_matcher.h
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/font_table_matcher_unittest.cc
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/font_unique_name_lookup.cc
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/font_unique_name_lookup.h
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/font_unique_name_lookup_unittest.cc
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/font_unique_name_table.proto
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/icu_fold_case_util.cc
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/icu_fold_case_util.h
[add] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/browser/font_unique_name_lookup/icu_fold_case_util_unittest.cc
[modify] https://crrev.com/7e00fed99a45512b17c974482c73ddcb72181850/content/test/BUILD.gn

Comment 2 by drott@chromium.org, Aug 1

Status: Fixed (was: S)
►

Sign in to add a comment