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

Issue 553203002: 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=181683

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1414 lines, -17 lines) Patch
M LayoutTests/TestExpectations View 1 chunk +1375 lines, -15 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-09 22:35:13 UTC) #1
eae
6 years, 3 months ago (2014-09-09 22:35:43 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as r181683.

Powered by Google App Engine
This is Rietveld 408576698