Compute a more correct "screen scope" transform for SVGSVGElement

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 432 matching lines @@
 SVGTransformTearOff* SVGSVGElement::createSVGTransform() {
   return SVGTransformTearOff::create(SVGTransform::create(kSvgTransformMatrix),
                                      0, PropertyIsNotAnimVal);
 }
 
 SVGTransformTearOff* SVGSVGElement::createSVGTransformFromMatrix(
     SVGMatrixTearOff* matrix) {
   return SVGTransformTearOff::create(matrix);
 }
 
-AffineTransform SVGSVGElement::localCoordinateSpaceTransform(
-    SVGElement::CTMScope mode) const {
-  AffineTransform viewBoxTransform;
-  if (!hasEmptyViewBox()) {
-    FloatSize size = currentViewportSize();
-    viewBoxTransform = viewBoxToViewTransform(size.width(), size.height());
-  }
-
+AffineTransform SVGSVGElement::localCoordinateSpaceTransform() const {
   AffineTransform transform;
   if (!isOutermostSVGSVGElement()) {
     SVGLengthContext lengthContext(this);
     transform.translate(m_x->currentValue()->value(lengthContext),
                         m_y->currentValue()->value(lengthContext));
-  } else if (mode == SVGElement::ScreenScope) {
-    if (LayoutObject* layoutObject = this->layoutObject()) {
-      FloatPoint location;
-      float zoomFactor = 1;
-
-      // At the SVG/HTML boundary (aka LayoutSVGRoot), we apply the
-      // localToBorderBoxTransform to map an element from SVG viewport
-      // coordinates to CSS box coordinates.  LayoutSVGRoot's localToAbsolute
-      // method expects CSS box coordinates.  We also need to adjust for the
-      // zoom level factored into CSS coordinates (bug #96361).
-      if (layoutObject->isSVGRoot()) {
-        location = toLayoutSVGRoot(layoutObject)
-                       ->localToBorderBoxTransform()
-                       .mapPoint(location);
-        zoomFactor = 1 / layoutObject->style()->effectiveZoom();
-      }
-
-      // Translate in our CSS parent coordinate space
-      // FIXME: This doesn't work correctly with CSS transforms.
-      location = layoutObject->localToAbsolute(location, UseTransforms);
-      location.scale(zoomFactor, zoomFactor);
-
-      // Be careful here! localToBorderBoxTransform() included the x/y offset
-      // coming from the viewBoxToViewTransform(), so we have to subtract it
-      // here (original cause of bug #27183)
-      transform.translate(location.x() - viewBoxTransform.e(),
-                          location.y() - viewBoxTransform.f());
-
-      // Respect scroll offset.
-      if (FrameView* view = document().view()) {
-        LayoutSize scrollOffset(view->getScrollOffset());
-        scrollOffset.scale(zoomFactor);
-        transform.translate(-scrollOffset.width(), -scrollOffset.height());
-      }
-    }
   }
-
-  return transform.multiply(viewBoxTransform);
+  if (!hasEmptyViewBox()) {
+    FloatSize size = currentViewportSize();
+    transform.multiply(viewBoxToViewTransform(size.width(), size.height()));
+  }
+  return transform;
 }
 
 bool SVGSVGElement::layoutObjectIsNeeded(const ComputedStyle& style) {
   // FIXME: We should respect display: none on the documentElement svg element
   // but many things in FrameView and SVGImage depend on the LayoutSVGRoot when
   // they should instead depend on the LayoutView.
   // https://bugs.webkit.org/show_bug.cgi?id=103493
   if (document().documentElement() == this)
     return true;
 
@@ skipping 249 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