Fix geometry mapping issues for float under inline

third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.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/html/HTMLIFrameElement.h"
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/LayoutTreeAsText.h"
 #include "core/layout/api/LayoutViewItem.h"
 #include "core/paint/ObjectPaintProperties.h"
 #include "core/paint/PaintPropertyTreePrinter.h"
@@ skipping 3202 matching lines @@
   auto* div = document().getElementById("div");
   auto* properties = div->layoutObject()->paintProperties()->overflowClip();
   EXPECT_EQ(FloatRect(0, 0, 7, 6), properties->clipRect().rect());
 
   div->setAttribute(HTMLNames::styleAttr, "");
   document().view()->updateAllLifecyclePhases();
   EXPECT_TRUE(!div->layoutObject()->paintProperties() ||
               !div->layoutObject()->paintProperties()->overflowClip());
 }
 
+TEST_P(PaintPropertyTreeBuilderTest, FloatUnderInline) {
+  setBodyInnerHTML(
+      "<span id='span'"
+      "    style='position: relative; top: 100px; left: 200px; opacity: 0.5'>"
+      "  <div id='target' style='float: left; width: 33px; height: 44px'>"
+      "  </div>"
+      "</span>");
+
+  LayoutObject* span = getLayoutObjectByElementId("span");
+  const auto* effect = span->paintProperties()->effect();
+  ASSERT_TRUE(effect);
+  EXPECT_EQ(0.5f, effect->opacity());
+
+  LayoutObject* target = getLayoutObjectByElementId("target");
+  const auto* localBorderBoxProperties =
+      target->paintProperties()->localBorderBoxProperties();
+  ASSERT_TRUE(localBorderBoxProperties);
+  EXPECT_EQ(LayoutPoint(8, 8), localBorderBoxProperties->paintOffset);

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

* { margin: 0 }

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

Added an absolute position div container to ensure we will just skip paint
offsets of the inline parent, but not paint offset of the containing block.

+  EXPECT_EQ(effect, localBorderBoxProperties->propertyTreeState.effect());
+}
+
 // A basic sanity check for over-invalidation of paint properties.
 TEST_P(PaintPropertyTreeBuilderTest, NoPaintPropertyUpdateOnBackgroundChange) {
   setBodyInnerHTML("<div id='div' style='background-color: blue'>DIV</div>");
   auto* div = document().getElementById("div");
 
   document().view()->updateAllLifecyclePhases();
   div->setAttribute(HTMLNames::styleAttr, "background-color: green");
   document().view()->updateLifecycleToLayoutClean();
   EXPECT_FALSE(div->layoutObject()->needsPaintPropertyUpdate());
 }
 
 }  // namespace blink