[SVG] SVGAnimatedBoolean migration to new SVG property impl.

Source/core/svg/SVGBoolean.h

 /*
  * 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 10 matching lines @@
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * 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.
  */
 
-#ifndef SVGLengthList_h
-#define SVGLengthList_h
+#ifndef SVGBoolean_h
+#define SVGBoolean_h
 
-#include "bindings/v8/ScriptWrappable.h"
-#include "core/svg/SVGLength.h"
-#include "core/svg/properties/NewSVGListPropertyHelper.h"
+#include "core/svg/properties/NewSVGProperty.h"
 
 namespace WebCore {
 
-class SVGLengthListTearOff;
+class SVGBoolean : public NewSVGPropertyBase {
+public:
+    // SVGBoolean do not have a tear-off type.

[haraken, 2014/01/09 11:09:23]

do not => does not

[kouhei (in TOK), 2014/01/10 08:16:10]

Done.

+    // The below typedef is used by NewSVGAnimatedProperty.
+    typedef void TearOffType;
+    typedef bool PrimitiveType;
 
-class SVGLengthList : public NewSVGListPropertyHelper<SVGLengthList, SVGLength> {
-public:
-    typedef SVGLengthListTearOff TearOffType;
-
-    static PassRefPtr<SVGLengthList> create(SVGLengthMode mode = LengthModeOther)
+    static PassRefPtr<SVGBoolean> create(bool value = false)
     {
-        return adoptRef(new SVGLengthList(mode));
+        return adoptRef(new SVGBoolean(value));
     }
 
-    ~SVGLengthList();
+    PassRefPtr<SVGBoolean> clone() const { return create(m_value); }
+    virtual PassRefPtr<NewSVGPropertyBase> cloneForAnimation(const String&) const OVERRIDE;
 
-    PassRefPtr<SVGLengthList> clone();
-
+    virtual String valueAsString() const OVERRIDE;
     void setValueAsString(const String&, ExceptionState&);
 
-    // NewSVGPropertyBase:
-    virtual PassRefPtr<NewSVGPropertyBase> cloneForAnimation(const String&) const OVERRIDE;
-    virtual String valueAsString() const OVERRIDE;
-
     virtual void add(PassRefPtr<NewSVGPropertyBase>, SVGElement*) OVERRIDE;
-    virtual void calculateAnimatedValue(SVGAnimationElement*, float percentage, unsigned repeatCount, PassRefPtr<NewSVGPropertyBase> fromValue, PassRefPtr<NewSVGPropertyBase> toValue, PassRefPtr<NewSVGPropertyBase> toAtEndOfDurationValue, SVGElement*) OVERRIDE;
+    virtual void calculateAnimatedValue(SVGAnimationElement*, float percentage, unsigned repeatCount, PassRefPtr<NewSVGPropertyBase> from, PassRefPtr<NewSVGPropertyBase> to, PassRefPtr<NewSVGPropertyBase> toAtEndOfDurationValue, SVGElement*) OVERRIDE;
     virtual float calculateDistance(PassRefPtr<NewSVGPropertyBase> to, SVGElement*) OVERRIDE;
 
-    static AnimatedPropertyType classType() { return AnimatedLengthList; }
+    bool operator==(const SVGBoolean& other) const { return m_value == other.m_value; }
+    bool operator!=(const SVGBoolean& other) const { return !operator==(other); }
+
+    bool value() const { return m_value; }
+    void setValue(bool value) { m_value = value; }
+
+    static AnimatedPropertyType classType() { return AnimatedBoolean; }
 
 private:
-    explicit SVGLengthList(SVGLengthMode);
+    SVGBoolean(bool value)
+        : NewSVGPropertyBase(classType())
+        , m_value(value)
+    {
+    }
 
-    bool adjustFromToListValues(PassRefPtr<SVGLengthList> fromList, PassRefPtr<SVGLengthList> toList, float percentage, bool isToAnimation, bool resizeAnimatedListIfNeeded);
+    bool m_value;
+};
 
-    template <typename CharType>
-    void parseInternal(const CharType*& ptr, const CharType* end, ExceptionState&);
-
-    SVGLengthMode m_mode;
-};
+inline PassRefPtr<SVGBoolean> toSVGBoolean(PassRefPtr<NewSVGPropertyBase> passBase)

[haraken, 2014/01/09 11:09:23]

It looks a bit strange that toXXX takes a PassRefPtr and returns a PassRefPtr. I
think normally toXXX takes a raw pointer and returns a raw pointer.

[kouhei (in TOK), 2014/01/10 08:16:10]

Discussed offline. Keeping this.

+{
+    RefPtr<NewSVGPropertyBase> base = passBase;
+    ASSERT(base->type() == SVGBoolean::classType());
+    return static_pointer_cast<SVGBoolean>(base.release());
+}
 
 } // namespace WebCore
 
-#endif // SVGLengthList_h
+#endif // SVGBoolean_h