Remove LayoutSVGViewportContainer's calcViewport and pointIsInsideViewportClip

third_party/WebKit/Source/core/svg/SVGSVGElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2006 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2010 Rob Buis <buis@kde.org>
  * Copyright (C) 2007 Apple Inc. All rights reserved.
  * Copyright (C) 2014 Google, Inc.
  *
  * 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 263 matching lines @@
     } else {
       invalidateSVGPresentationAttributeStyle();
       setNeedsStyleRecalc(LocalStyleChange,
                           StyleChangeReasonForTracing::fromAttribute(attrName));
     }
   }
 
   if (SVGFitToViewBox::isKnownAttribute(attrName)) {
     updateRelativeLengthsOrViewBox = true;
     invalidateRelativeLengthClients();
-    if (LayoutObject* object = layoutObject())
-      object->setNeedsTransformUpdate();
   }
 
   if (updateRelativeLengthsOrViewBox ||
       SVGZoomAndPan::isKnownAttribute(attrName)) {
     SVGElement::InvalidationGuard invalidationGuard(this);
-    if (layoutObject())
-      markForLayoutAndParentResourceInvalidation(layoutObject());
+    if (auto* layoutObject = this->layoutObject()) {
+      markForLayoutAndParentResourceInvalidation(layoutObject);
+      layoutObject->setNeedsBoundariesUpdate();

[fs, 2016/12/13 10:11:23]

This is a bugfix? (Well, I guess the next line is too...) Is it coupled with the
TODO in LayoutSVGViewportContainer?

[pdr., 2016/12/13 19:48:55]

This was me jumping the gun on that TODO. I've reverted the changes in this
file.

+      layoutObject->setNeedsTransformUpdate();
+    }
     return;
   }
 
   SVGGraphicsElement::svgAttributeChanged(attrName);
 }
 
 // FloatRect::intersects does not consider horizontal or vertical lines (because
 // of isEmpty()).
 static bool intersectsAllowingEmpty(const FloatRect& r1, const FloatRect& r2) {
   if (r1.width() < 0 || r1.height() < 0 || r2.width() < 0 || r2.height() < 0)
@@ skipping 457 matching lines @@
   visitor->trace(m_width);
   visitor->trace(m_height);
   visitor->trace(m_translation);
   visitor->trace(m_timeContainer);
   visitor->trace(m_viewSpec);
   SVGGraphicsElement::trace(visitor);
   SVGFitToViewBox::trace(visitor);
 }
 
 }  // namespace blink