Use outlineBox if we have an outline or shadow

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 2b6ae7a1b380aa24cd0a05fc8ac3a1fc2914f332..29ce2ee7614ddfe821a337cf5d74c3bba14601e7 100644
--- a/Source/core/rendering/RenderObject.cpp
+++ b/Source/core/rendering/RenderObject.cpp
@@ -1514,7 +1514,10 @@ bool RenderObject::repaintAfterLayoutIfNeeded(const RenderLayerModelObject* repa
         // This ASSERT fails due to animations.  See https://bugs.webkit.org/show_bug.cgi?id=37048
         // ASSERT(!newOutlineBoxRectPtr || *newOutlineBoxRectPtr == outlineBoundsForRepaint(repaintContainer));
         newOutlineBox = newOutlineBoxRectPtr ? *newOutlineBoxRectPtr : outlineBoundsForRepaint(repaintContainer);
-        if (newOutlineBox.location() != oldOutlineBox.location() || (mustRepaintBackgroundOrBorder() && (newBounds != oldBounds || newOutlineBox != oldOutlineBox)))
+
+        bool hasOutlineBox = hasOutline() || (style()->boxShadow() && style()->boxShadow()->shadows().size() > 0);

[Julien - ping for review, 2014/03/07 00:01:10]

style()->boxShadow() && style()->boxShadow()->shadows().size() > 0 looks like it
could be moved to RenderStyle::hasBoxShadow() to match other getters.

Maybe we want hasOutlineOrBoxShadow as a new getter?

[dsinclair, 2014/03/07 03:18:56]

Done.

+        if ((hasOutlineBox && newOutlineBox.location() != oldOutlineBox.location())
+            || (mustRepaintBackgroundOrBorder() && (newBounds != oldBounds || (hasOutlineBox && newOutlineBox != oldOutlineBox))))
             fullRepaint = true;
     }