[SCM] WebKit Debian packaging branch, debian/unstable, updated. debian/1.1.15-1-40151-g37bb677

hyatt hyatt at 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Sat Sep 26 08:37:35 UTC 2009


The following commit has been merged in the debian/unstable branch:
commit 7af2f3b8b127022817bc3635e040284c5fa98dbc
Author: hyatt <hyatt at 268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Wed Apr 28 19:52:35 2004 +0000

    	Cut the time spent on an operation inside widthForNextCharacter from 17% of the function time down to less than
    	5% merely by adding a check for non-zero letter-spacing (thus avoiding double precision math in the
    	common case where we just add 0 between letters).
    
            Reviewed by rjw
    
            * WebCoreSupport.subproj/WebTextRenderer.m:
            (widthForNextCharacter):
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@6505 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebKit/ChangeLog b/WebKit/ChangeLog
index 9ada97c..55170cb 100644
--- a/WebKit/ChangeLog
+++ b/WebKit/ChangeLog
@@ -1,3 +1,14 @@
+2004-04-27  David Hyatt  <hyatt at apple.com>
+
+	Cut the time spent on an operation inside widthForNextCharacter from 17% of the function time down to less than 
+	5% merely by adding a check for non-zero letter-spacing (thus avoiding double precision math in the 
+	common case where we just add 0 between letters).
+	
+        Reviewed by rjw
+
+        * WebCoreSupport.subproj/WebTextRenderer.m:
+        (widthForNextCharacter):
+
 2004-04-28  John Sullivan  <sullivan at apple.com>
 
         Initial plumbing to get the font panel to be updated from an editable
diff --git a/WebKit/WebCoreSupport.subproj/WebTextRenderer.m b/WebKit/WebCoreSupport.subproj/WebTextRenderer.m
index fa1c01b..69cbffa 100644
--- a/WebKit/WebCoreSupport.subproj/WebTextRenderer.m
+++ b/WebKit/WebCoreSupport.subproj/WebTextRenderer.m
@@ -1919,7 +1919,7 @@ static float widthForNextCharacter(CharacterWidthIterator *iterator, ATSGlyphRef
     }
     
     // Account for letter-spacing
-    if (width > 0)
+    if (iterator->style->letterSpacing && width > 0)
         width += iterator->style->letterSpacing;
 
     // Account for padding.  khtml uses space padding to justify text.  We

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list