Remove LayoutSVGViewportContainer's calcViewport and pointIsInsideViewportClip

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

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp
index fe065b243022de74ecde5e120d40c83670130034..8554c6bf3bce3bf03356ba5c9c739fd6df1eac5f 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGContainer.cpp
@@ -46,9 +46,6 @@ void LayoutSVGContainer::layout() {
   ASSERT(needsLayout());
   LayoutAnalyzer::Scope analyzer(*this);
 
-  // Allow LayoutSVGViewportContainer to update its viewport.
-  calcViewport();
-
   // Allow LayoutSVGTransformableContainer to update its transform.

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

Nit: Maybe update this comment too (like in the header.)

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

Done. Went with a generic "Update the local transform in subclasses".

   SVGTransformChange transformChange = calculateLocalTransform();
   m_didScreenScaleFactorChange =
@@ -182,10 +179,6 @@ void LayoutSVGContainer::updateCachedBoundaries() {
 bool LayoutSVGContainer::nodeAtFloatPoint(HitTestResult& result,
                                           const FloatPoint& pointInParent,
                                           HitTestAction hitTestAction) {
-  // Give LayoutSVGViewportContainer a chance to apply its viewport clip
-  if (!pointIsInsideViewportClip(pointInParent))
-    return false;
-
   FloatPoint localPoint;
   if (!SVGLayoutSupport::transformToUserSpaceAndCheckClipping(
           *this, localToSVGParentTransform(), pointInParent, localPoint))