Remove LayoutSVGViewportContainer's calcViewport and pointIsInsideViewportClip

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

Index: third_party/WebKit/Source/core/svg/SVGSVGElement.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGSVGElement.cpp b/third_party/WebKit/Source/core/svg/SVGSVGElement.cpp
index 1f85592c1822a16924f7609ad33c997bd092c3ec..d39e93c3b0b2574e8e81bbd463f0410c5d605ddb 100644
--- a/third_party/WebKit/Source/core/svg/SVGSVGElement.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGSVGElement.cpp
@@ -281,15 +281,16 @@ void SVGSVGElement::svgAttributeChanged(const QualifiedName& 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;
   }