Flip for writing mode exactly along the containing block chain.

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

Index: third_party/WebKit/Source/core/layout/LayoutObject.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutObject.cpp b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
index ccef0e4d686fbe0047f7d4cccab686dc226671cd..555cedbe8a143998d35cac3fadceeae3fa93a525 100644
--- a/third_party/WebKit/Source/core/layout/LayoutObject.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
@@ -1576,11 +1576,14 @@ bool LayoutObject::mapToVisualRectInAncestorSpace(const LayoutBoxModelObject* an
     if (LayoutObject* parent = this->parent()) {
         if (parent->isBox()) {
             LayoutBox* parentBox = toLayoutBox(parent);
-            if (!parentBox->mapScrollingContentsRectToBoxSpace(rect, parent == ancestor ? ApplyNonScrollOverflowClip : ApplyOverflowClip, visualRectFlags))
-                return false;
+
             // Never flip for SVG as it handles writing modes itself.
-            if (!isSVG())
+            if (!isSVG()) {
                 parentBox->flipForWritingMode(rect);
+            }

[wkorman, 2016/07/18 23:21:28]

nit: don't need the curlies

[chrishtr, 2016/07/18 23:26:57]

Removed.

+
+            if (!parentBox->mapScrollingContentsRectToBoxSpace(rect, parent == ancestor ? ApplyNonScrollOverflowClip : ApplyOverflowClip, visualRectFlags))
+                return false;
         }
         return parent->mapToVisualRectInAncestorSpace(ancestor, rect, visualRectFlags);
     }