Don't use a magic value for 'auto' oriented <marker>s

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

 /*
  * Copyright (C) 2004, 2005, 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) Research In Motion Limited 2009-2010. 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.
  *
@@ skipping 72 matching lines @@
 {
     SVGMarkerElement* marker = toSVGMarkerElement(element());
     ASSERT(marker);
 
     SVGLengthContext lengthContext(marker);
     return FloatPoint(marker->refX()->currentValue()->value(lengthContext), marker->refY()->currentValue()->value(lengthContext));
 }
 
 float LayoutSVGResourceMarker::angle() const
 {
-    ASSERT(element());
-
-    float angle = -1;
-    if (orientType() == SVGMarkerOrientAngle)
-        angle = toSVGMarkerElement(*element()).orientAngle()->currentValue()->value();
-
-    return angle;
+    return toSVGMarkerElement(element())->orientAngle()->currentValue()->value();
 }
 
 SVGMarkerUnitsType LayoutSVGResourceMarker::markerUnits() const
 {
     return toSVGMarkerElement(element())->markerUnits()->currentValue()->enumValue();
 }
 
 SVGMarkerOrientType LayoutSVGResourceMarker::orientType() const
 {
     return toSVGMarkerElement(element())->orientType()->currentValue()->enumValue();
 }
 
 AffineTransform LayoutSVGResourceMarker::markerTransformation(const FloatPoint& origin, float autoAngle, float strokeWidth) const
 {
-    float markerAngle = angle();
     bool useStrokeWidth = markerUnits() == SVGMarkerUnitsStrokeWidth;
 
     AffineTransform transform;
     transform.translate(origin.x(), origin.y());
-    transform.rotate(markerAngle == -1 ? autoAngle : markerAngle);
+    transform.rotate(orientType() == SVGMarkerOrientAngle ? angle() : autoAngle);
     transform = markerContentTransformation(transform, referencePoint(), useStrokeWidth ? strokeWidth : -1);
     return transform;
 }
 
 AffineTransform LayoutSVGResourceMarker::markerContentTransformation(const AffineTransform& contentTransformation, const FloatPoint& origin, float strokeWidth) const
 {
     // The 'origin' coordinate maps to SVGs refX/refY, given in coordinates relative to the viewport established by the marker
     FloatPoint mappedOrigin = viewportTransform().mapPoint(origin);
 
     AffineTransform transformation = contentTransformation;
@@ skipping 28 matching lines @@
 
 bool LayoutSVGResourceMarker::calculateLocalTransform()
 {
     // TODO(fs): Temporarily, needing a layout implies that the local transform
     // has changed. This should be updated to be more precise and factor in the
     // actual (relevant) changes to the computed user-space transform.
     return selfNeedsLayout();
 }
 
 } // namespace blink