Fix geometry mapping issues for float under inline

third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp
index eb4b6f3c3f839f616a27e5fc3664a55fe1748483..851020393ffa38593bef40777a2a7e0a4571c706 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp
@@ -70,6 +70,11 @@ void LayoutSVGBlock::willBeDestroyed() {
   LayoutBlockFlow::willBeDestroyed();
 }
 
+void LayoutSVGBlock::updateFromStyle() {
+  LayoutBlockFlow::updateFromStyle();
+  setFloating(false);

[wkorman, 2017/01/04 22:27:08]

What was causing floating to be set to true in some cases? Is this just
enforcing something already in spec that we were not enforcing to date, and dev
could just put a float style on an svg element and it would have taken?

[Xianzhu, 2017/01/04 22:49:59]

isFloating() is set in LayoutBox::updateFromStyle():
  setFloating(!isOutOfFlowPositioned() && styleToUse.isFloating());

Most SVG objects ignores 'float' because they are not LayoutBox. LayoutSVGBlock
inherits from LayoutBlockFlow so it get isFloating set if there is 'float' style
which should be ignored.

Most of our code already ignore isFloating() of an LayoutSVGBlock, except for
PaintPropertyTreeBuilder in this CL
(https://codereview.chromium.org/2575423003/diff/160001/third_party/WebKit/Sou...).

+}
+
 void LayoutSVGBlock::styleDidChange(StyleDifference diff,
                                     const ComputedStyle* oldStyle) {
   if (diff.needsFullLayout()) {