Clean up in-flow position offset for LayoutInline mapping to visual rect.

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

Index: third_party/WebKit/Source/core/layout/LayoutInline.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutInline.cpp b/third_party/WebKit/Source/core/layout/LayoutInline.cpp
index 71782d3e40ceebd5141612c58f2a19d3739b6da5..38fea136385ad37b28c2e9df045cd046af39a920 100644
--- a/third_party/WebKit/Source/core/layout/LayoutInline.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutInline.cpp
@@ -1078,20 +1078,14 @@ bool LayoutInline::mapToVisualRectInAncestorSpace(const LayoutBoxModelObject* an
     if (!container)
         return true;
 
-    LayoutPoint topLeft = rect.location();
-
     if (style()->hasInFlowPosition() && layer()) {
         // Apply the in-flow position offset when invalidating a rectangle. The layer
         // is translated, but the layout box isn't, so we need to do this to get the
         // right dirty rect. Since this is called from LayoutObject::setStyle, the relative position
         // flag on the LayoutObject has been cleared, so use the one on the style().
-        topLeft += layer()->offsetForInFlowPosition();
+        rect.move(layer()->offsetForInFlowPosition());
     }
 
-    // FIXME: We ignore the lightweight clipping rect that controls use, since if |o| is in mid-layout,

[wkorman, 2016/07/21 00:44:51]

This comment looks old and isn't clearly still relevant, so I vote for deleting.

-    // its controlClipRect will be wrong. For overflow clip we use the values cached by the layer.
-    rect.setLocation(topLeft);
-
     LayoutBox* containerBox = container->isBox() ? toLayoutBox(container) : nullptr;
     if (containerBox && !containerBox->mapScrollingContentsRectToBoxSpace(rect, container == ancestor ? ApplyNonScrollOverflowClip : ApplyOverflowClip, visualRectFlags))
         return false;