Title: [125189] trunk/Source/WebCore
Revision
125189
Author
[email protected]
Date
2012-08-09 11:41:05 -0700 (Thu, 09 Aug 2012)

Log Message

[chromium] Enable kerning on Android
https://bugs.webkit.org/show_bug.cgi?id=92737

Patch by Iain Merrick <[email protected]> on 2012-08-09
Reviewed by Adam Barth.

This #ifdef was incorrectly copied from our forked WebKit repository
while unforking our Android-specific changes. Kerning works without
any known problems.

Covered by existing tests (layout and instrumentation tests).

* platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp:
(WebCore::ComplexTextController::ComplexTextController):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (125188 => 125189)


--- trunk/Source/WebCore/ChangeLog	2012-08-09 18:27:46 UTC (rev 125188)
+++ trunk/Source/WebCore/ChangeLog	2012-08-09 18:41:05 UTC (rev 125189)
@@ -1,3 +1,19 @@
+2012-08-09  Iain Merrick  <[email protected]>
+
+        [chromium] Enable kerning on Android
+        https://bugs.webkit.org/show_bug.cgi?id=92737
+
+        Reviewed by Adam Barth.
+
+        This #ifdef was incorrectly copied from our forked WebKit repository
+        while unforking our Android-specific changes. Kerning works without
+        any known problems.
+
+        Covered by existing tests (layout and instrumentation tests).
+
+        * platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp:
+        (WebCore::ComplexTextController::ComplexTextController):
+
 2012-08-09  'Pavel Feldman'  <[email protected]>
 
         Web Inspector: follow up to r125174 - fix subtype use.

Modified: trunk/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp (125188 => 125189)


--- trunk/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp	2012-08-09 18:27:46 UTC (rev 125188)
+++ trunk/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp	2012-08-09 18:41:05 UTC (rev 125189)
@@ -80,11 +80,6 @@
     m_item.string = m_normalizedBuffer.get();
     m_item.stringLength = m_normalizedBufferLength;
 
-#if OS(ANDROID)
-    // Kerning does not currently work on Android.
-    m_item.shaperFlags = HB_ShaperFlag_NoKerning;
-#endif
-
     reset(startingX);
     m_startingY = startingY;
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to