Fix geometry mapping issues for float under inline

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

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/LayoutView.h"
 #include "core/layout/PaintInvalidationState.h"
 #include "core/paint/PaintLayer.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 679 matching lines @@
 
   LayoutRect targetVisualRect = target->localVisualRect();
   EXPECT_EQ(LayoutRect(0, 0, 400, 400), targetVisualRect);
   LayoutRect rect = targetVisualRect;
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(&layoutView(), rect));
   EXPECT_EQ(LayoutRect(0, 0, 200, 200), rect);
   checkPaintInvalidationStateRectMapping(rect, targetVisualRect, *target,
                                          layoutView(), layoutView());
 }
 
+TEST_F(VisualRectMappingTest, FloatUnderInline) {
+  setBodyInnerHTML(
+      "<span style='position: relative; top: 100px; left: 200px'>"
+      "  <div id='target' style='float: left; width: 33px; height: 44px'>"
+      "  </div>"
+      "</span>");
+
+  LayoutBox* target = toLayoutBox(getLayoutObjectByElementId("target"));
+
+  LayoutRect targetVisualRect = target->localVisualRect();
+  EXPECT_EQ(LayoutRect(0, 0, 33, 44), targetVisualRect);
+  LayoutRect rect = targetVisualRect;
+  EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(&layoutView(), rect));
+  EXPECT_EQ(LayoutRect(8, 8, 33, 44), rect);

[chrishtr, 2016/12/16 22:10:30]

add * { margin: 0 } to get rid of the 8px

[Xianzhu, 2016/12/17 05:43:18]

Added an position:absolute container to test we accumulate block ancestor paint
offset for floating objects.

+  checkPaintInvalidationStateRectMapping(rect, targetVisualRect, *target,
+                                         layoutView(), layoutView());
+}
+
 }  // namespace blink