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

Unified Diff: third_party/WebKit/Source/platform/fonts/win/FontPlatformDataWin.cpp

Issue 1930503003: Remove font subpixel positioning flag (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Retain test-only handling on linux Created 4 years, 8 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
« no previous file with comments | « third_party/WebKit/Source/platform/fonts/win/FontCacheSkiaWin.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/platform/fonts/win/FontPlatformDataWin.cpp
diff --git a/third_party/WebKit/Source/platform/fonts/win/FontPlatformDataWin.cpp b/third_party/WebKit/Source/platform/fonts/win/FontPlatformDataWin.cpp
index 9e7e6c04bdc4df7ab595975ac2bfdf7f0f761346..285cb0b6c44caffbbf24a4fb4abef30f2e11549b 100644
--- a/third_party/WebKit/Source/platform/fonts/win/FontPlatformDataWin.cpp
+++ b/third_party/WebKit/Source/platform/fonts/win/FontPlatformDataWin.cpp
@@ -62,11 +62,10 @@ void FontPlatformData::setupPaint(SkPaint* paint, float, const Font*) const
if (ts >= m_minSizeForAntiAlias) {
- if (m_useSubpixelPositioning
- // Disable subpixel text for certain older fonts at smaller sizes as
- // they tend to get quite blurry at non-integer sizes and positions.
- // For high-DPI this workaround isn't required.
- && (ts >= m_minSizeForSubpixel
+ // Disable subpixel text for certain older fonts at smaller sizes as
+ // they tend to get quite blurry at non-integer sizes and positions.
+ // For high-DPI this workaround isn't required.
+ if ((ts >= m_minSizeForSubpixel
|| FontCache::fontCache()->deviceScaleFactor() >= 1.5)
// Subpixel text positioning looks pretty bad without font
@@ -117,14 +116,9 @@ static int computePaintTextFlags(String fontFamilyName)
}
-void FontPlatformData::querySystemForRenderStyle(bool)
+void FontPlatformData::querySystemForRenderStyle()
{
m_paintTextFlags = computePaintTextFlags(fontFamilyName());
}
-bool FontPlatformData::defaultUseSubpixelPositioning()
-{
- return FontCache::fontCache()->useSubpixelPositioning();
-}
-
} // namespace blink
« no previous file with comments | « third_party/WebKit/Source/platform/fonts/win/FontCacheSkiaWin.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698