Skip to content
  • rjw's avatar
    WebCore: · 9409ba5b
    rjw authored
            Fixed 3133261.  This fix really has two parts.  This first part
            fixes the stupid stack allocated buffer that caused the crash.
            The second part makes any RenderText with a large string perform
            much faster.  The page mentioned in the bug used to load and draw
            in about 15 minutes.  Now it loads in about 10 seconds and draws in about
            2 seconds.  The performance optimization caches a widths array
            for the string in the RenderText, and only updates that array if
            the font or text for the RenderText change.
    
            Reviewed by john.
    
            * khtml/rendering/render_text.cpp:
            (RenderText::RenderText):
            (RenderText::setStyle):
            (RenderText::~RenderText):
            (RenderText::computeWidths):
            (RenderText::widthFromBuffer):
            (RenderText::trimmedMinMaxWidth):
            (RenderText::calcMinMaxWidth):
            (RenderText::setText):
            (RenderText::width):
            * khtml/rendering/render_text.h:
    
    WebKit:
            Fixed 3133261.  This fix really has two parts.  This first part
            is here in WebTextRenderer.  The second part adds some width
            caching to RenderText.  I was using a stack allocated array,
            this would blow out the stack for large strings.
    
            Reviewed by john.
    
            * WebCoreSupport.subproj/WebTextRenderer.m:
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3152 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    9409ba5b