Strip anonymous blocks when change in style removes need for them

Source/core/rendering/RenderBlock.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2007 David Smith (catfish.man@gmail.com)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 310 matching lines @@
     return oldStyle->borderTopWidth() != newStyle->borderTopWidth()
         || oldStyle->borderBottomWidth() != newStyle->borderBottomWidth()
         || oldStyle->paddingTop() != newStyle->paddingTop()
         || oldStyle->paddingBottom() != newStyle->paddingBottom();
 }
 
 void RenderBlock::styleDidChange(StyleDifference diff, const RenderStyle* oldStyle)
 {
     RenderBox::styleDidChange(diff, oldStyle);
 
+    if (isFloatingOrOutOfFlowPositioned() && (oldStyle && !oldStyle->isFloating() && !oldStyle->hasOutOfFlowPosition()) && parent() && parent()->isRenderBlockFlow())

[esprehn, 2014/05/19 20:54:58]

You don't need these nested parens, it's all &&

+        toRenderBlock(parent())->removeAnonymousWrappersIfRequired();
+
     RenderStyle* newStyle = style();
 
     if (!isAnonymousBlock()) {
         // Ensure that all of our continuation blocks pick up the new style.
         for (RenderBlock* currCont = blockElementContinuation(); currCont; currCont = currCont->blockElementContinuation()) {
             RenderBoxModelObject* nextCont = currCont->continuation();
             currCont->setContinuation(0);
             currCont->setStyle(newStyle);
             currCont->setContinuation(nextCont);
         }
@@ skipping 704 matching lines @@
         return false;
 
     if (!prev || !next)
         return true;
 
     // Make sure the types of the anonymous blocks match up.
     return prev->isAnonymousColumnsBlock() == next->isAnonymousColumnsBlock()
            && prev->isAnonymousColumnSpanBlock() == next->isAnonymousColumnSpanBlock();
 }
 
+void RenderBlock::removeAnonymousWrappersIfRequired()
+{
+    RenderBlock* child = toRenderBlock(firstChildBox());

[esprehn, 2014/05/19 20:54:58]

How do you know this cast is safe?

+    while (child && (child->isAnonymousBlock() || child->isFloatingOrOutOfFlowPositioned())) {
+        RenderBlock* next = toRenderBlock(child->nextSiblingBox());

[esprehn, 2014/05/19 20:54:58]

ditto

+        // A continuation means the wrappers are still required.
+        if (next && next->continuation())
+            return;
+        if (child->isAnonymousBlock())
+            collapseAnonymousBlockChild(this, child);
+        // |child| may have been destroyed.
+        child = next;
+    }
+}
+
 void RenderBlock::collapseAnonymousBlockChild(RenderBlock* parent, RenderBlock* child)
 {
     // It's possible that this block's destruction may have been triggered by the
     // child's removal. Just bail if the anonymous child block is already being
     // destroyed. See crbug.com/282088
     if (child->beingDestroyed())
         return;
     parent->setNeedsLayoutAndPrefWidthsRecalc();
     parent->setChildrenInline(child->childrenInline());
     RenderObject* nextSibling = child->nextSibling();
@@ skipping 3927 matching lines @@
 void RenderBlock::showLineTreeAndMark(const InlineBox* markedBox1, const char* markedLabel1, const InlineBox* markedBox2, const char* markedLabel2, const RenderObject* obj) const
 {
     showRenderObject();
     for (const RootInlineBox* root = firstRootBox(); root; root = root->nextRootBox())
         root->showLineTreeAndMark(markedBox1, markedLabel1, markedBox2, markedLabel2, obj, 1);
 }
 
 #endif
 
 } // namespace WebCore