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

Issue 556313002: Merge 181492 "[DirectWrite] Disable subpixel font positioning fo..." (Closed)

Created:
6 years, 3 months ago by eae
Modified:
6 years, 3 months ago
Reviewers:
eae
CC:
blink-reviews, jamesr, krit, jbroman, danakj, Rik, Stephen Chennney, pdr., rwlbuis
Project:
blink
Visibility:
Public.

Description

Merge 181492 "[DirectWrite] Disable subpixel font positioning fo..." > [DirectWrite] Disable subpixel font positioning for certain fonts/sizes > > Conditionally disable subpixel text for certain older fonts at sizes > below 16px in order to improve legibility and rendering quality. This is > limited to a specific set of fonts and does not apply to webfonts. > This list of fonts in question includes all fonts in the "Microsoft Core > fonts for the Web" collection. > > Specifically the following fonts: arial, courier new, times new roman, > andale mono, comic sans, impact, georgia, trebuchet, verdana, webdings, > tahoma, and lucida console. > > R=cpu@chromium.org, scottmg@chromium.org > BUG=405445, 395425 > > Review URL: https://codereview.chromium.org/544843004 TBR=eae@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181739

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1215 lines, -6 lines) Patch
M LayoutTests/TestExpectations View 2 chunks +1176 lines, -4 lines 0 comments Download
M LayoutTests/fast/text/sub-pixel/text-scaling-webfont.html View 1 chunk +1 line, -1 line 0 comments Download
M Source/platform/fonts/harfbuzz/FontPlatformDataHarfBuzz.h View 2 chunks +3 lines, -0 lines 0 comments Download
M Source/platform/fonts/harfbuzz/FontPlatformDataHarfBuzz.cpp View 7 chunks +7 lines, -0 lines 0 comments Download
M Source/platform/fonts/win/FontCacheSkiaWin.cpp View 1 chunk +27 lines, -0 lines 0 comments Download
M Source/platform/fonts/win/FontPlatformDataWin.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
eae
6 years, 3 months ago (2014-09-10 16:02:30 UTC) #1
eae
6 years, 3 months ago (2014-09-10 16:03:18 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as r181739.

Powered by Google App Engine
This is Rietveld 408576698