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

Issue 339693002: [DirectWrite] Force fully aliased text for certain fonts at small sizes (Closed)

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

Description

[DirectWrite] Force fully aliased text for certain fonts at small sizes Force the use of fully aliased text for certain common east asian fonts at smaller font sizes to ensure that text is legible. This is only required for fonts without an embedded bitmap at a the specific size and the minimum size for enabling anti aliasing is set on a per font basis. BUG=383090, 383040 R=dglazkov@chromium.org, cpu@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176244

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -14 lines) Patch
M LayoutTests/platform/win/fast/forms/datetimelocal/datetimelocal-appearance-l10n-expected.png View 1 Binary file 0 comments Download
M LayoutTests/platform/win/fast/forms/month/month-appearance-l10n-expected.png View 1 Binary file 0 comments Download
M LayoutTests/platform/win/fast/text/justify-ideograph-complex-expected.png View 1 Binary file 0 comments Download
M LayoutTests/platform/win/fast/text/justify-ideograph-simple-expected.png View 1 Binary file 0 comments Download
M LayoutTests/platform/win/fast/text/justify-ideograph-vertical-expected.png View 1 Binary file 0 comments Download
M LayoutTests/platform/win/media/track/track-cue-rendering-vertical-expected.png View 1 Binary file 0 comments Download
M LayoutTests/platform/win/virtual/antialiasedtext/fast/text/international/wrap-CJK-001-expected.png View 1 Binary file 0 comments Download
M LayoutTests/platform/win/virtual/antialiasedtext/fast/text/justify-ideograph-complex-expected.png View 1 Binary file 0 comments Download
M LayoutTests/platform/win/virtual/antialiasedtext/fast/text/justify-ideograph-simple-expected.png View 1 Binary file 0 comments Download
M LayoutTests/platform/win/virtual/antialiasedtext/fast/text/justify-ideograph-vertical-expected.png View 1 Binary file 0 comments Download
M Source/platform/fonts/win/FontCacheSkiaWin.cpp View 1 chunk +17 lines, -0 lines 0 comments Download
M Source/platform/fonts/win/FontPlatformDataWin.cpp View 1 chunk +16 lines, -14 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
eae
6 years, 6 months ago (2014-06-16 17:54:04 UTC) #1
dglazkov
lgtm
6 years, 6 months ago (2014-06-16 18:23:30 UTC) #2
eae
The CQ bit was checked by eae@chromium.org
6 years, 6 months ago (2014-06-16 20:24:06 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/eae@chromium.org/339693002/20001
6 years, 6 months ago (2014-06-16 20:24:45 UTC) #4
eae
6 years, 6 months ago (2014-06-16 21:38:18 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r176244 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698