Fix synchronization of SVGAnimatedAngle (<marker orient>)

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

 /*
  * Copyright (C) 2014 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "core/svg/SVGAnimatedAngle.h"
 
 #include "core/SVGNames.h"
-#include "core/svg/SVGAngleTearOff.h"
-#include "core/svg/SVGMarkerElement.h"
 
 namespace blink {
 
-SVGAnimatedAngle::SVGAnimatedAngle(SVGMarkerElement* contextElement)
+SVGAnimatedAngle::SVGAnimatedAngle(SVGElement* contextElement)
     : SVGAnimatedProperty<SVGAngle>(contextElement, SVGNames::orientAttr, SVGAngle::create())
     , m_orientType(SVGAnimatedEnumeration<SVGMarkerOrientType>::create(contextElement, SVGNames::orientAttr, baseValue()->orientType()))
 {
 }
 
 SVGAnimatedAngle::~SVGAnimatedAngle()
 {
 }
 
 DEFINE_TRACE(SVGAnimatedAngle)
 {
     visitor->trace(m_orientType);
     SVGAnimatedProperty<SVGAngle>::trace(visitor);
 }
 
+bool SVGAnimatedAngle::needsSynchronizeAttribute()
+{
+    return m_orientType->needsSynchronizeAttribute()
+        || SVGAnimatedProperty<SVGAngle>::needsSynchronizeAttribute();
+}
+
 void SVGAnimatedAngle::synchronizeAttribute()
 {
-    DEFINE_STATIC_LOCAL(const AtomicString, autoValue, ("auto", AtomicString::ConstructFromLiteral));
-    AtomicString value;
-    if (m_orientType->currentValue()->enumValue() == SVGMarkerOrientAuto)
-        value = autoValue;
-    else
-        value = AtomicString(currentValue()->valueAsString());
-
-    contextElement()->setSynchronizedLazyAttribute(attributeName(), value);
+    // If the current value is not an <angle> we synchronize the value of the
+    // wrapped enumeration.
+    if (m_orientType->currentValue()->enumValue() != SVGMarkerOrientAngle) {
+        m_orientType->synchronizeAttribute();
+        return;
+    }
+    SVGAnimatedProperty<SVGAngle>::synchronizeAttribute();
 }
 
 void SVGAnimatedAngle::setAnimatedValue(PassRefPtrWillBeRawPtr<SVGPropertyBase> value)
 {
     SVGAnimatedProperty<SVGAngle>::setAnimatedValue(value);
     m_orientType->setAnimatedValue(currentValue()->orientType());
 }
 
 void SVGAnimatedAngle::animationEnded()
 {
     SVGAnimatedProperty<SVGAngle>::animationEnded();
     m_orientType->animationEnded();
 }
 
 } // namespace blink