Remove SVGPathElement.pathSegList and related interfaces

third_party/WebKit/Source/core/svg/SVGPath.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 SVGNumberList_h
-#define SVGNumberList_h
+#ifndef SVGPath_h
+#define SVGPath_h
 
-#include "bindings/core/v8/ScriptWrappable.h"
-#include "core/svg/SVGNumber.h"
-#include "core/svg/properties/SVGListPropertyHelper.h"
+#include "core/svg/properties/SVGAnimatedProperty.h"
 
 namespace blink {
 
-class SVGNumberListTearOff;
+class SVGPathByteStream;
 
-class SVGNumberList final : public SVGListPropertyHelper<SVGNumberList, SVGNumber> {
+class SVGPath : public SVGPropertyBase {
 public:
-    typedef SVGNumberListTearOff TearOffType;
+    typedef void TearOffType;

[pdr., 2015/10/23 04:40:55]

Is this needed?

[fs, 2015/10/23 09:12:33]

Not required, but the alternative would be to have a lot of explicit <..., void>
specifications, so doing it this way is nicer I think.

 
-    static PassRefPtrWillBeRawPtr<SVGNumberList> create()
+    static PassRefPtrWillBeRawPtr<SVGPath> create()
     {
-        return adoptRefWillBeNoop(new SVGNumberList());
+        return adoptRefWillBeNoop(new SVGPath());
     }
 
-    ~SVGNumberList() override;
+    ~SVGPath() override;
 
-    void setValueAsString(const String&, ExceptionState&);
+    const SVGPathByteStream& byteStream() const;
+    SVGPathByteStream& mutableByteStream();
 
     // SVGPropertyBase:
+    PassRefPtrWillBeRawPtr<SVGPath> clone() const;
+    PassRefPtrWillBeRawPtr<SVGPropertyBase> cloneForAnimation(const String&) const override;
     String valueAsString() const override;
+    void setValueAsString(const String&, ExceptionState&);
 
     void add(PassRefPtrWillBeRawPtr<SVGPropertyBase>, SVGElement*) override;
     void calculateAnimatedValue(SVGAnimationElement*, float percentage, unsigned repeatCount, PassRefPtrWillBeRawPtr<SVGPropertyBase> fromValue, PassRefPtrWillBeRawPtr<SVGPropertyBase> toValue, PassRefPtrWillBeRawPtr<SVGPropertyBase> toAtEndOfDurationValue, SVGElement*) override;
     float calculateDistance(PassRefPtrWillBeRawPtr<SVGPropertyBase> to, SVGElement*) override;
 
-    static AnimatedPropertyType classType() { return AnimatedNumberList; }
-
-    Vector<float> toFloatVector() const;
+    static AnimatedPropertyType classType() { return AnimatedPath; }
 
 private:
-    SVGNumberList();
+    SVGPath();
+    explicit SVGPath(PassOwnPtr<SVGPathByteStream>);
 
-    template <typename CharType>
-    bool parse(const CharType*& ptr, const CharType* end);
+    mutable OwnPtr<SVGPathByteStream> m_byteStream;

[pdr., 2015/10/23 04:40:55]

Does this need to be mutable?

[fs, 2015/10/23 09:12:33]

Should cancel out with the const_cast, so no. Dropped.

 };
 
+inline PassRefPtrWillBeRawPtr<SVGPath> toSVGPath(PassRefPtrWillBeRawPtr<SVGPropertyBase> passBase)
+{
+    RefPtrWillBeRawPtr<SVGPropertyBase> base = passBase;
+    ASSERT(base->type() == SVGPath::classType());
+    return static_pointer_cast<SVGPath>(base.release());
+}
+
 } // namespace blink
 
-#endif // SVGNumberList_h
+#endif