[svg2] Make 'width' and 'height' presentation attributes

Source/core/svg/SVGLengthContext.cpp

 /*
  * Copyright (C) 2004, 2005, 2006 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007 Rob Buis <buis@kde.org>
  * Copyright (C) 2007 Apple Inc. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. 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.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "config.h"
 #include "core/svg/SVGLengthContext.h"
 
 #include "core/css/CSSHelper.h"
+#include "core/css/CSSPrimitiveValue.h"
 #include "core/layout/LayoutObject.h"
 #include "core/layout/style/LayoutStyle.h"
 #include "core/svg/SVGSVGElement.h"
 #include "platform/LengthFunctions.h"
 #include "platform/fonts/FontMetrics.h"
 
 namespace blink {
 
 static inline float dimensionForLengthMode(SVGLengthMode mode, const FloatSize& viewportSize)
 {
     switch (mode) {
     case SVGLengthMode::Width:
         return viewportSize.width();
     case SVGLengthMode::Height:
         return viewportSize.height();
     case SVGLengthMode::Other:
         return sqrtf(viewportSize.diagonalLengthSquared() / 2);
     }
     ASSERT_NOT_REACHED();
     return 0;
 }
 
 static float convertValueFromPercentageToUserUnits(const SVGLength& value, const FloatSize& viewportSize)
 {
-    return value.scaleByPercentage(dimensionForLengthMode(value.unitMode(), viewportSize));
+    return CSSPrimitiveValue::clampToCSSLengthRange(value.scaleByPercentage(dimensionForLengthMode(value.unitMode(), viewportSize)));
 }
 
 SVGLengthContext::SVGLengthContext(const SVGElement* context)
     : m_context(context)
 {
 }
 
 FloatRect SVGLengthContext::resolveRectangle(const SVGElement* context, SVGUnitTypes::SVGUnitType type, const FloatRect& viewport, const SVGLength& x, const SVGLength& y, const SVGLength& width, const SVGLength& height)
 {
     ASSERT(type != SVGUnitTypes::SVG_UNIT_TYPE_UNKNOWN);
@@ skipping 62 matching lines @@
 }
 
 float SVGLengthContext::valueForLength(const Length& length, float zoom, float dimension)
 {
     ASSERT(zoom != 0);
     return floatValueForLength(length, dimension * zoom) / zoom;
 }
 
 float SVGLengthContext::convertValueToUserUnits(float value, SVGLengthMode mode, SVGLengthType fromUnit) const
 {
+    float userUnits = value;
     switch (fromUnit) {
     case LengthTypeUnknown:
         return 0;
+    case LengthTypePX:
     case LengthTypeNumber:
-        return value;
-    case LengthTypePX:
-        return value;
+        userUnits = value;
+        break;
     case LengthTypePercentage: {
         FloatSize viewportSize;
         if (!determineViewport(viewportSize))
             return 0;
-        return value * dimensionForLengthMode(mode, viewportSize) / 100;
+        userUnits = value * dimensionForLengthMode(mode, viewportSize) / 100;
+        break;
     }
     case LengthTypeEMS:
-        return convertValueFromEMSToUserUnits(value);
+        userUnits = convertValueFromEMSToUserUnits(value);
+        break;
     case LengthTypeEXS:
-        return convertValueFromEXSToUserUnits(value);
+        userUnits = convertValueFromEXSToUserUnits(value);
+        break;
     case LengthTypeCM:
-        return value * cssPixelsPerCentimeter;
+        userUnits = value * cssPixelsPerCentimeter;
+        break;
     case LengthTypeMM:
-        return value * cssPixelsPerMillimeter;
+        userUnits = value * cssPixelsPerMillimeter;
+        break;
     case LengthTypeIN:
-        return value * cssPixelsPerInch;
+        userUnits = value * cssPixelsPerInch;
+        break;
     case LengthTypePT:
-        return value * cssPixelsPerPoint;
+        userUnits = value * cssPixelsPerPoint;
+        break;
     case LengthTypePC:
-        return value * cssPixelsPerPica;
+        userUnits = value * cssPixelsPerPica;
+        break;
+    default:
+        ASSERT_NOT_REACHED();
+        break;
     }
 
-    ASSERT_NOT_REACHED();
-    return 0;
+    // Since we mix css <length> values with svg's length values we need to
+    // clamp values to the narrowest range, otherwise it can result in
+    // rendering issues.
+    return CSSPrimitiveValue::clampToCSSLengthRange(userUnits);
 }
 
 float SVGLengthContext::convertValueFromUserUnits(float value, SVGLengthMode mode, SVGLengthType toUnit) const
 {
     switch (toUnit) {
     case LengthTypeUnknown:
         return 0;
     case LengthTypeNumber:
         return value;
     case LengthTypePercentage: {
@@ skipping 108 matching lines @@
 
     const SVGSVGElement& svg = toSVGSVGElement(*viewportElement);
     viewportSize = svg.currentViewBoxRect().size();
     if (viewportSize.isEmpty())
         viewportSize = svg.currentViewportSize();
 
     return true;
 }
 
 }