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

Unified Diff: ui/gfx/font_render_params_linux.cc

Issue 387743002: Apply hinting in SkiaTextRenderer::SetFontRenderParams(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: change autohinting default Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/gfx/font_render_params_linux.cc
diff --git a/ui/gfx/font_render_params_linux.cc b/ui/gfx/font_render_params_linux.cc
index 9d75f38af92deea24cc3736a1ac9c151c4db7989..4a06c59f13a957098855de933fc620776c6e4f7c 100644
--- a/ui/gfx/font_render_params_linux.cc
+++ b/ui/gfx/font_render_params_linux.cc
@@ -63,6 +63,9 @@ void LoadDefaults(FontRenderParams* params, bool renderer) {
}
#if defined(OS_LINUX) && !defined(OS_CHROMEOS)
+ // TODO(derat): Get the autohinter setting from FontConfig. Until then,
+ // maintain backward compatibility with what we were doing previously.
+ params->autohinter = renderer;
Daniel Erat 2014/07/10 23:46:35 so this is pretty gross. SkiaTextRenderer wasn't p
msw 2014/07/11 00:29:09 Acknowledged.
const LinuxFontDelegate* delegate = LinuxFontDelegate::instance();
if (delegate) {
params->antialiasing = delegate->UseAntialiasing();

Powered by Google App Engine
This is Rietveld 408576698