SVGSMILElement should respect SVGTests results

Source/core/svg/SVGAnimationElement.h

 /*
  * Copyright (C) 2004, 2005 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006 Rob Buis <buis@kde.org>
  * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Cameron McCormack <cam@mcc.id.au>
  * 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
@@ skipping 42 matching lines @@
 enum CalcMode {
     CalcModeDiscrete,
     CalcModeLinear,
     CalcModePaced,
     CalcModeSpline
 };
 
 class SVGAnimationElement : public SVGSMILElement,
                             public SVGTests {
 public:
+    virtual bool isValid() const OVERRIDE FINAL { return SVGTests::isValid(); }

[fs, 2014/07/29 08:21:11]

Nit: Couldn't this be made private access?

+
     // SVGAnimationElement
     float getStartTime() const;
     float getCurrentTime() const;
     float getSimpleDuration() const;
 
     void beginElement();
     void beginElementAt(float offset);
     void endElement();
     void endElementAt(float offset);
 
@@ skipping 131 matching lines @@
     String m_lastValuesAnimationFrom;
     String m_lastValuesAnimationTo;
     bool m_hasInvalidCSSAttributeType;
     CalcMode m_calcMode;
     AnimationMode m_animationMode;
 };
 
 } // namespace blink
 
 #endif // SVGAnimationElement_h