Get rid of anonymous table* wrappers when they are no longer needed.

Get rid of anonymous table* wrappers when they are no longer needed.

When a renderer is removed, don't just see if an anonymous parent
should be destroyed, but keep looking further up in the ancestry
as well.

BUG=389129
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182082

[mstensho (USE GERRIT), 2014-09-03 20:08:10]

mstensho@opera.com changed reviewers:
+ jchaffraix@chromium.org

[Julien - ping for review, 2014-09-08 21:28:49]

https://codereview.chromium.org/540453003/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderObject.cpp (right):

https://codereview.chromium.org/540453003/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderObject.cpp:2817: if (destroyRoot != this &&
!destroyRootParent->isTableCell() && !destroyRootParent->isTableRow() &&
!destroyRootParent->isTableSection() && !destroyRootParent->isTable())
This seems very artificial to limit this to table and table parts. I agree with
you that they are the only example where we will walk more than a single level
but this may change and thus I don't think we should special-case table.

[mstensho (USE GERRIT), 2014-09-16 13:00:07]

https://codereview.chromium.org/540453003/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderObject.cpp (right):

https://codereview.chromium.org/540453003/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderObject.cpp:2817: if (destroyRoot != this &&
!destroyRootParent->isTableCell() && !destroyRootParent->isTableRow() &&
!destroyRootParent->isTableSection() && !destroyRootParent->isTable())
On 2014/09/08 21:28:49, Julien Chaffraix - PST wrote:
> This seems very artificial to limit this to table and table parts. I agree
with
> you that they are the only example where we will walk more than a single level
> but this may change and thus I don't think we should special-case table.

Done.

[Julien - ping for review, 2014-09-16 17:40:49]

The CQ bit was checked by jchaffraix@chromium.org

[Julien - ping for review, 2014-09-16 17:40:50]

lgtm

[commit-bot: I haz the power, 2014-09-16 17:41:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/540453003/40001

[commit-bot: I haz the power, 2014-09-16 18:44:07]

Committed patchset #3 (id:40001) as 182082