<rdar://problem/9354979> REGRESSION (r83070-r83126): Conversation takes 10 seconds...
authorDavid Hyatt <hyatt@apple.com>
Thu, 5 May 2011 20:04:01 +0000 (20:04 +0000)
committerAdemar de Souza Reis Jr <ademar.reis@openbossa.org>
Thu, 30 Jun 2011 17:45:57 +0000 (14:45 -0300)
commit6f7f62f4fa45217da4dba868c80c7d715674bcc3
tree25dbcf42e227a6f2929a223837c48d1a32bff938
parenta52dbae362a295e0adfb7ee4fdc21734ae4b7b45
<rdar://problem/9354979> REGRESSION (r83070-r83126): Conversation takes 10 seconds to load and makes mail unresponsive

Reviewed by Darin Adler.

Culled inlines were triggering some pathological line box tree groveling that isn't even necessary.
Removed the ancient code (that used to be in RenderFlow), since it made no sense in the RenderBlock case
(it was running for inline blocks, which was definitely not even the intent) or in the RenderInline case
(the object being removed has no effect on any lines).

Also tweaked culledInlineFirstLineBox and culledInlineLastLineBox to avoid bailing if the first replaced object that
is encountered has a null inlineBoxWrapper().  Just a slight speed optimization to avoid an extra null check.

* rendering/RenderBlock.cpp:
(WebCore::RenderBlock::destroy):
* rendering/RenderInline.cpp:
(WebCore::RenderInline::destroy):
(WebCore::RenderInline::culledInlineFirstLineBox):
(WebCore::RenderInline::culledInlineLastLineBox):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@85869 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderBlock.cpp
Source/WebCore/rendering/RenderInline.cpp