Fix geometry mapping issues for float under inline

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

 /*
  * Copyright (C) 2006 Apple Computer, Inc.
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 52 matching lines @@
   // This code path should never be taken for SVG, as we're assuming
   // useTransforms=true everywhere, absoluteQuads should be used.
   ASSERT_NOT_REACHED();
 }
 
 void LayoutSVGBlock::willBeDestroyed() {
   SVGResourcesCache::clientDestroyed(this);
   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()) {
     setNeedsBoundariesUpdate();
     if (diff.transformChanged())
       setNeedsTransformUpdate();
   }
 
   if (isBlendingAllowed()) {
     bool hasBlendModeChanged =
@@ skipping 56 matching lines @@
 
 bool LayoutSVGBlock::nodeAtPoint(HitTestResult&,
                                  const HitTestLocation&,
                                  const LayoutPoint&,
                                  HitTestAction) {
   ASSERT_NOT_REACHED();
   return false;
 }
 
 }  // namespace blink