Let mapToVisualRectInAncestorSpace apply ancestor clip if not scrollsOverflow

third_party/WebKit/Source/core/layout/LayoutBox.cpp

Index: third_party/WebKit/Source/core/layout/LayoutBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBox.cpp b/third_party/WebKit/Source/core/layout/LayoutBox.cpp
index 9655aac8a07ee74ede97abbb5aa82cc18d1b38e5..b8c19d92a1701b8c25e2559f749f9d8a6cc2b8ea 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBox.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBox.cpp
@@ -1989,8 +1989,10 @@ bool LayoutBox::mapToVisualRectInAncestorSpace(const LayoutBoxModelObject* ances
     if (container->hasOverflowClip()) {
         LayoutBox* containerBox = toLayoutBox(container);
         containerBox->mapScrollingContentsRectToBoxSpace(rect);
-        if (container != ancestor && !containerBox->applyOverflowClip(rect, visualRectFlags))
-            return false;
+        if (container != ancestor || !containerBox->scrollsOverflow()) {

[pdr., 2016/04/07 18:23:45]

This block (check hasOverflowClip , mapScrollingContentsRectToBoxSpace, etc) is
repeated 3x verbatim. Can we refactor this logic into a single function,
possibly on LayoutBox?

[Xianzhu, 2016/04/07 19:04:50]

Done.

+            if (!containerBox->applyOverflowClip(rect, visualRectFlags))
+                return false;
+        }
     }
 
     if (ancestorSkipped) {