Don't force full repaint on size change in style

Source/core/rendering/RenderObject.cpp

Index: Source/core/rendering/RenderObject.cpp
diff --git a/Source/core/rendering/RenderObject.cpp b/Source/core/rendering/RenderObject.cpp
index 9ca93e6cb026aae6ef61fc27d699913bdaea63a5..ed1a847e9800066e23378472cf1d997da19c9b38 100644
--- a/Source/core/rendering/RenderObject.cpp
+++ b/Source/core/rendering/RenderObject.cpp
@@ -1748,7 +1748,7 @@ bool RenderObject::repaintAfterLayoutIfNeeded(const RenderLayerModelObject* repa
         repaintUsingContainer(repaintContainer, pixelSnappedIntRect(oldBounds.x(), newBounds.maxY(), oldBounds.width(), -deltaBottom), invalidationReason);
 
     // FIXME: This is a limitation of our visual overflow being a single rectangle.
-    if (!style()->boxShadow() && !style()->hasBorderImageOutsets() && !style()->hasOutline())
+    if (!style()->hasBorder() && !style()->boxShadow() && !style()->hasBorderImageOutsets() && !style()->hasOutline())

[Julien - ping for review, 2014/06/11 17:53:24]

The border does not generate visual overflow and is already included into the
clippedOverflowRectForRepaint above so this change looks suspicious. Could you
explain why it is needed?

[Xianzhu, 2014/06/16 16:25:29]

For example, when an object with border is increasing its width, we need paint
not only the new right part, but also the old right border (to erase it).
Without this change, we'll only paint the right part, not the old right border.

Noticed this because without this change several layout tests fail because of
under-repaint of right or bottom borders.

[Julien - ping for review, 2014/06/16 20:04:24]

OK, I really think we should include the border into the regular incremental
invalidation above. The code path below should be used for _visual effects_
(which doesn't include borders).

[Xianzhu, 2014/06/16 20:49:16]

I had tried to modify regular invalidation code, but it seemed duplicate with
the code below about border. The code path below does handle border and seems to
work well for my example case. Could we reuse the code below also for border?

If we need to separate border and visual effects, should be remove
border-related code below?

         return false;
 
     // We didn't move, but we did change size. Invalidate the delta, which will consist of possibly