RenderBlock and RenderInline have confusingly named object creation methods: RenderBlock::createRootBox/createRootInlineBox RenderInline::createFlowBox/createInlineFlowBox where the 2nd method in both cases just calls the first and then appends the created object, i.e., is not really a different object creation method, and doesn't create different objects. Also confusing because there are no "RootBox" or "FlowBox" classes.
Created attachment 32032 [details] patch: rename methods to createAndAppend...()
Comment on attachment 32032 [details] patch: rename methods to createAndAppend...() Looks great!
(In reply to comment #2) > (From update of attachment 32032 [details] [review]) > Looks great! Thanks! Could you please commit the patch for me?
There were tab characters in the ChangeLog, and the ChangeLog was missing the bug URL. I corrected these when landing.
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/rendering/RenderBlock.cpp M WebCore/rendering/RenderBlock.h M WebCore/rendering/RenderInline.cpp M WebCore/rendering/RenderInline.h M WebCore/rendering/RenderSVGInline.cpp M WebCore/rendering/RenderSVGInline.h M WebCore/rendering/RenderSVGText.cpp M WebCore/rendering/RenderSVGText.h M WebCore/rendering/bidi.cpp Committed r45517 M WebCore/ChangeLog M WebCore/rendering/RenderSVGInline.h M WebCore/rendering/RenderSVGText.h M WebCore/rendering/RenderInline.cpp M WebCore/rendering/RenderBlock.cpp M WebCore/rendering/bidi.cpp M WebCore/rendering/RenderInline.h M WebCore/rendering/RenderBlock.h M WebCore/rendering/RenderSVGText.cpp M WebCore/rendering/RenderSVGInline.cpp r45517 = a2815d0bab937dd4b3bfc42ffd13b5d7a3b7a0fc (trunk) No changes between current HEAD and refs/remotes/trunk Resetting to the latest refs/remotes/trunk http://trac.webkit.org/changeset/45517