Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(15)

Issue 2265373002: Require certain faimilies to contain certain styles (Closed)

Created:
4 years, 4 months ago by Ilya Kulshin
Modified:
4 years, 4 months ago
Reviewers:
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@2758
Target Ref:
refs/pending/branch-heads/2758
Project:
chromium
Visibility:
Public.

Description

Require certain faimilies to contain certain styles This is a workaround for 635932, so that Chrome will ignore certain font families unless they contain the specified font styles. We start out requiring that Helvetica and Open Sans contain the regular variant, because in some cases they get installed only in specialized variants which results in fonts incorrectly displaying bold or condensed. BUG=635932 Review-Url: https://codereview.chromium.org/2259733002 Cr-Commit-Position: refs/heads/master@{#413323} (cherry picked from commit 0bf300d07fe226dc48da9cbfaa6d9ea36480f66d) Committed: https://chromium.googlesource.com/chromium/src/+/8708aee2406f5bd80ea6a13093a85959d0af9ed9

Patch Set 1 #

Patch Set 2 : Require certain faimilies to contain certain styles #

Unified diffs Side-by-side diffs Delta from patch set Stats (+102 lines, -2 lines) Patch
M content/browser/renderer_host/dwrite_font_proxy_message_filter_win.cc View 3 chunks +75 lines, -1 line 0 comments Download
M tools/metrics/histograms/histograms.xml View 2 chunks +27 lines, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
Ilya Kulshin
4 years, 4 months ago (2016-08-22 21:58:55 UTC) #2
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
8708aee2406f5bd80ea6a13093a85959d0af9ed9.

Powered by Google App Engine
This is Rietveld 408576698