[RAL] Fix nested non-composited RenderView repainting

Source/core/rendering/RenderBox.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. 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
@@ skipping 1570 matching lines @@
     // ASSERT(&newRepaintContainer == containerForRepaint());
 
     const LayoutRect oldRepaintRect = previousRepaintRect();
     const LayoutPoint oldPositionFromRepaintContainer = previousPositionFromRepaintContainer();
     setPreviousRepaintRect(clippedOverflowRectForRepaint(&newRepaintContainer));
     setPreviousPositionFromRepaintContainer(positionFromRepaintContainer(&newRepaintContainer));
 
     // If we are set to do a full repaint that means the RenderView will be
     // invalidated. We can then skip issuing of invalidations for the child
     // renderers as they'll be covered by the RenderView.
-    if (view()->doingFullRepaint() && this != view()) {
+    if (view()->doingFullRepaint()) {
         LayoutStateMaintainer statePusher(*this, isTableRow() ? LayoutSize() : locationOffset());

[Julien - ping for review, 2014/05/30 02:39:32]

It is weird that we push LayoutState if we don't expect to invalidate (your
patch adds some invalidation though).

[leviw_travelin_and_unemployed, 2014/05/30 15:52:57]

We still need to generate our cached repaint rects. LayoutState makes that
faster.

         RenderObject::repaintTreeAfterLayout(newRepaintContainer);
         return;
     }
 
     if ((onlyNeededPositionedMovementLayout() && compositingState() != PaintsIntoOwnBacking)
         || (shouldDoFullRepaintIfSelfPaintingLayer()
             && hasLayer()
             && layer()->isSelfPaintingLayer())) {
         setShouldDoFullRepaintAfterLayout(true);
     }
@@ skipping 3085 matching lines @@
         return 0;
 
     if (!layoutState && !flowThreadContainingBlock())
         return 0;
 
     RenderBlock* containerBlock = containingBlock();
     return containerBlock->offsetFromLogicalTopOfFirstPage() + logicalTop();
 }
 
 } // namespace WebCore