Title: [96635] trunk/Source/WebCore
Revision
96635
Author
carycl...@google.com
Date
2011-10-04 13:20:13 -0700 (Tue, 04 Oct 2011)

Log Message

Enable sub pixel text (Skia on Mac)
https://bugs.webkit.org/show_bug.cgi?id=69354
http://code.google.com/p/chromium/issues/detail?id=98165

Reviewed by Eric Seidel.

No new tests. Layout test pixel results have been
disabled and will be rebaselined once all Skia on Mac
changes are in.

* platform/graphics/skia/FontSkia.cpp:
(WebCore::setupPaint):
Enable sub pixel text. This is set by default to enabled
by CoreGraphics, but by default disabled on Skia.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (96634 => 96635)


--- trunk/Source/WebCore/ChangeLog	2011-10-04 20:16:51 UTC (rev 96634)
+++ trunk/Source/WebCore/ChangeLog	2011-10-04 20:20:13 UTC (rev 96635)
@@ -1,3 +1,20 @@
+2011-10-04  Cary Clark  <carycl...@google.com>
+
+        Enable sub pixel text (Skia on Mac)
+        https://bugs.webkit.org/show_bug.cgi?id=69354
+        http://code.google.com/p/chromium/issues/detail?id=98165
+
+        Reviewed by Eric Seidel.
+
+        No new tests. Layout test pixel results have been
+        disabled and will be rebaselined once all Skia on Mac
+        changes are in.
+
+        * platform/graphics/skia/FontSkia.cpp:
+        (WebCore::setupPaint):
+        Enable sub pixel text. This is set by default to enabled
+        by CoreGraphics, but by default disabled on Skia.
+
 2011-10-04  Anders Carlsson  <ander...@apple.com>
 
         Remove PlatformWheelEvent::m_isAccepted

Modified: trunk/Source/WebCore/platform/graphics/skia/FontSkia.cpp (96634 => 96635)


--- trunk/Source/WebCore/platform/graphics/skia/FontSkia.cpp	2011-10-04 20:16:51 UTC (rev 96634)
+++ trunk/Source/WebCore/platform/graphics/skia/FontSkia.cpp	2011-10-04 20:20:13 UTC (rev 96635)
@@ -89,6 +89,7 @@
     paint->setTextSkewX(platformData.m_syntheticOblique ? -SK_Scalar1 / 4 : 0);
     paint->setAutohinted(false); // freetype specific
     paint->setLCDRenderText(shouldSmoothFonts);
+    paint->setSubpixelText(true);
 }
 
 // TODO: This needs to be split into helper functions to better scope the
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to