Title: [94303] trunk/Source/WebCore
Revision
94303
Author
[email protected]
Date
2011-09-01 10:20:29 -0700 (Thu, 01 Sep 2011)

Log Message

Typesetting features need not force complex for single-character runs
https://bugs.webkit.org/show_bug.cgi?id=66498

Typesetting features have no effect on a solitary character, so they shouldn't enable complex layout
unless multiple characters are present.

Patch by Ned Holbrook <[email protected]> on 2011-09-01
Reviewed by Dan Bernstein.

No new tests, this is merely a performance optimization.

* platform/graphics/Font.cpp:
(WebCore::Font::codePath): typesettingFeatures() only forces Complex for multiple-character runs.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (94302 => 94303)


--- trunk/Source/WebCore/ChangeLog	2011-09-01 17:15:46 UTC (rev 94302)
+++ trunk/Source/WebCore/ChangeLog	2011-09-01 17:20:29 UTC (rev 94303)
@@ -1,3 +1,18 @@
+2011-09-01  Ned Holbrook  <[email protected]>
+
+        Typesetting features need not force complex for single-character runs
+        https://bugs.webkit.org/show_bug.cgi?id=66498
+
+        Typesetting features have no effect on a solitary character, so they shouldn't enable complex layout
+        unless multiple characters are present.
+
+        Reviewed by Dan Bernstein.
+
+        No new tests, this is merely a performance optimization.
+
+        * platform/graphics/Font.cpp:
+        (WebCore::Font::codePath): typesettingFeatures() only forces Complex for multiple-character runs.
+
 2011-08-31  Jeff Miller  <[email protected]>
 
         Defer creating AVPlayer and AVPlayerItem in MediaPlayerPrivateAVFoundation::setPreload()

Modified: trunk/Source/WebCore/platform/graphics/Font.cpp (94302 => 94303)


--- trunk/Source/WebCore/platform/graphics/Font.cpp	2011-09-01 17:15:46 UTC (rev 94302)
+++ trunk/Source/WebCore/platform/graphics/Font.cpp	2011-09-01 17:20:29 UTC (rev 94303)
@@ -416,7 +416,7 @@
             return Complex;
     }
 
-    if (typesettingFeatures())
+    if (run.length() > 1 && typesettingFeatures())
         return Complex;
 
     return result;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to