Title: [222077] trunk/Source/WebCore
Revision
222077
Author
[email protected]
Date
2017-09-15 00:08:25 -0700 (Fri, 15 Sep 2017)

Log Message

[FreeType] Complex text is enabled too often after r221909
https://bugs.webkit.org/show_bug.cgi?id=176907

Reviewed by Sergio Villar Senin.

In r221909 we enabled complex text by default following the same cocoa ifdefs, but I forgot another ifdef in the
cpp file.

* platform/graphics/FontCascade.cpp:
(WebCore::FontCascade::codePath const):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (222076 => 222077)


--- trunk/Source/WebCore/ChangeLog	2017-09-15 05:10:37 UTC (rev 222076)
+++ trunk/Source/WebCore/ChangeLog	2017-09-15 07:08:25 UTC (rev 222077)
@@ -1,3 +1,16 @@
+2017-09-14  Carlos Garcia Campos  <[email protected]>
+
+        [FreeType] Complex text is enabled too often after r221909
+        https://bugs.webkit.org/show_bug.cgi?id=176907
+
+        Reviewed by Sergio Villar Senin.
+
+        In r221909 we enabled complex text by default following the same cocoa ifdefs, but I forgot another ifdef in the
+        cpp file.
+
+        * platform/graphics/FontCascade.cpp:
+        (WebCore::FontCascade::codePath const):
+
 2017-09-14  Commit Queue  <[email protected]>
 
         Unreviewed, rolling out r221932 and r221933.

Modified: trunk/Source/WebCore/platform/graphics/FontCascade.cpp (222076 => 222077)


--- trunk/Source/WebCore/platform/graphics/FontCascade.cpp	2017-09-15 05:10:37 UTC (rev 222076)
+++ trunk/Source/WebCore/platform/graphics/FontCascade.cpp	2017-09-15 07:08:25 UTC (rev 222077)
@@ -597,7 +597,7 @@
     if (s_codePath != Auto)
         return s_codePath;
 
-#if PLATFORM(COCOA)
+#if PLATFORM(COCOA) || USE(FREETYPE)
     // Because Font::applyTransforms() doesn't know which features to enable/disable in the simple code path, it can't properly handle feature or variant settings.
     // FIXME: https://bugs.webkit.org/show_bug.cgi?id=150791: @font-face features should also cause this to be complex.
     if (m_fontDescription.featureSettings().size() > 0 || !m_fontDescription.variantSettings().isAllNormal())
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to