Skip to content
  • jamesr@google.com's avatar
    2010-04-02 James Robinson <jamesr@chromium.org> · 19548ad3
    jamesr@google.com authored
            Reviewed by Darin Adler.
    
            Splits RenderBlock::layoutInline into smaller functions
            https://bugs.webkit.org/show_bug.cgi?id=36921
    
            RenderBlock::layoutInlineChildren is 351 lines long and very difficult
            to comprehend or edit safely. This patch splits it up into a few
            slightly smaller functions.  Most of the code is now in the 241 line
            layoutRunsAndFloats() which is a slight improvement.
    
            Perf neutral on the page cyclers.  This doesn't introduce any function
            calls into the hottest layout paths inside layoutRunsAndFloats and
            findNextLineBreak.
    
            No change in behavior, no new tests.
    
            * rendering/RenderBlock.h:
            (WebCore::RenderBlock::FloatWithRect::FloatWithRect):
            * rendering/RenderBlockLineLayout.cpp:
            (WebCore::RenderBlock::layoutReplacedElements):
            (WebCore::RenderBlock::createLineBoxesForResolver):
            (WebCore::RenderBlock::layoutRunsAndFloats):
            (WebCore::RenderBlock::layoutInlineChildren):
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@57030 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    19548ad3