Commit 1f771e50 authored by Henrique Ferreiro's avatar Henrique Ferreiro Committed by Commit Bot

Remove unused Create() from core/svg 3/3

As advised in [1], after porting all uses of Foo::Create() to
MakeGarbageCollected<Foo>() in //third_party/blink/renderer/core/svg,
this CL removes the unused factory methods.

[1] https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/iJ1bawbxbWs/vEdfT5QtBgAJ

Bug: 939691
Change-Id: I1785f70be35d35096d239dda1dabe99911e8f380
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1581619
Commit-Queue: Henrique Ferreiro <hferreiro@igalia.com>
Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#653598}
parent b7123a6c
...@@ -32,8 +32,6 @@ class SVGMPathElement final : public SVGElement, public SVGURIReference { ...@@ -32,8 +32,6 @@ class SVGMPathElement final : public SVGElement, public SVGURIReference {
USING_GARBAGE_COLLECTED_MIXIN(SVGMPathElement); USING_GARBAGE_COLLECTED_MIXIN(SVGMPathElement);
public: public:
DECLARE_NODE_FACTORY(SVGMPathElement);
explicit SVGMPathElement(Document&); explicit SVGMPathElement(Document&);
~SVGMPathElement() override; ~SVGMPathElement() override;
......
...@@ -44,10 +44,6 @@ class SVGNumber : public SVGPropertyHelper<SVGNumber> { ...@@ -44,10 +44,6 @@ class SVGNumber : public SVGPropertyHelper<SVGNumber> {
typedef SVGNumberTearOff TearOffType; typedef SVGNumberTearOff TearOffType;
typedef float PrimitiveType; typedef float PrimitiveType;
static SVGNumber* Create(float value = 0.0f) {
return MakeGarbageCollected<SVGNumber>(value);
}
explicit SVGNumber(float = 0.0f); explicit SVGNumber(float = 0.0f);
virtual SVGNumber* Clone() const; virtual SVGNumber* Clone() const;
...@@ -89,10 +85,6 @@ DEFINE_SVG_PROPERTY_TYPE_CASTS(SVGNumber); ...@@ -89,10 +85,6 @@ DEFINE_SVG_PROPERTY_TYPE_CASTS(SVGNumber);
// offset = "<number> | <percentage>" // offset = "<number> | <percentage>"
class SVGNumberAcceptPercentage final : public SVGNumber { class SVGNumberAcceptPercentage final : public SVGNumber {
public: public:
static SVGNumberAcceptPercentage* Create(float value = 0) {
return MakeGarbageCollected<SVGNumberAcceptPercentage>(value);
}
explicit SVGNumberAcceptPercentage(float = 0); explicit SVGNumberAcceptPercentage(float = 0);
SVGNumber* Clone() const override; SVGNumber* Clone() const override;
......
...@@ -44,10 +44,6 @@ class SVGNumberList final ...@@ -44,10 +44,6 @@ class SVGNumberList final
public: public:
typedef SVGNumberListTearOff TearOffType; typedef SVGNumberListTearOff TearOffType;
static SVGNumberList* Create() {
return MakeGarbageCollected<SVGNumberList>();
}
SVGNumberList(); SVGNumberList();
~SVGNumberList() override; ~SVGNumberList() override;
......
...@@ -41,14 +41,6 @@ class SVGNumberListTearOff final ...@@ -41,14 +41,6 @@ class SVGNumberListTearOff final
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
static SVGNumberListTearOff* Create(
SVGNumberList* target,
SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) {
return MakeGarbageCollected<SVGNumberListTearOff>(target, binding,
property_is_anim_val);
}
SVGNumberListTearOff(SVGNumberList* target, SVGNumberListTearOff(SVGNumberList* target,
SVGAnimatedPropertyBase* binding, SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) PropertyIsAnimValType property_is_anim_val)
......
...@@ -43,12 +43,6 @@ class SVGNumberOptionalNumber final : public SVGPropertyBase { ...@@ -43,12 +43,6 @@ class SVGNumberOptionalNumber final : public SVGPropertyBase {
typedef void TearOffType; typedef void TearOffType;
typedef void PrimitiveType; typedef void PrimitiveType;
static SVGNumberOptionalNumber* Create(SVGNumber* first_number,
SVGNumber* second_number) {
return MakeGarbageCollected<SVGNumberOptionalNumber>(first_number,
second_number);
}
SVGNumberOptionalNumber(SVGNumber* first_number, SVGNumber* second_number); SVGNumberOptionalNumber(SVGNumber* first_number, SVGNumber* second_number);
SVGNumberOptionalNumber* Clone() const; SVGNumberOptionalNumber* Clone() const;
......
...@@ -40,12 +40,6 @@ class SVGNumberTearOff : public SVGPropertyTearOff<SVGNumber> { ...@@ -40,12 +40,6 @@ class SVGNumberTearOff : public SVGPropertyTearOff<SVGNumber> {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
static SVGNumberTearOff* Create(SVGNumber* target,
SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) {
return MakeGarbageCollected<SVGNumberTearOff>(target, binding,
property_is_anim_val);
}
static SVGNumberTearOff* CreateDetached(); static SVGNumberTearOff* CreateDetached();
SVGNumberTearOff(SVGNumber*, SVGNumberTearOff(SVGNumber*,
......
...@@ -42,11 +42,6 @@ class SVGPath final : public SVGPropertyBase { ...@@ -42,11 +42,6 @@ class SVGPath final : public SVGPropertyBase {
public: public:
typedef void TearOffType; typedef void TearOffType;
static SVGPath* Create() { return MakeGarbageCollected<SVGPath>(); }
static SVGPath* Create(cssvalue::CSSPathValue* path_value) {
return MakeGarbageCollected<SVGPath>(path_value);
}
SVGPath(); SVGPath();
explicit SVGPath(cssvalue::CSSPathValue*); explicit SVGPath(cssvalue::CSSPathValue*);
~SVGPath() override; ~SVGPath() override;
......
...@@ -42,8 +42,6 @@ void SVGPathElement::Trace(blink::Visitor* visitor) { ...@@ -42,8 +42,6 @@ void SVGPathElement::Trace(blink::Visitor* visitor) {
SVGGeometryElement::Trace(visitor); SVGGeometryElement::Trace(visitor);
} }
DEFINE_NODE_FACTORY(SVGPathElement)
Path SVGPathElement::AttributePath() const { Path SVGPathElement::AttributePath() const {
return path_->CurrentValue()->GetStylePath()->GetPath(); return path_->CurrentValue()->GetStylePath()->GetPath();
} }
......
...@@ -33,8 +33,6 @@ class SVGPathElement final : public SVGGeometryElement { ...@@ -33,8 +33,6 @@ class SVGPathElement final : public SVGGeometryElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGPathElement);
explicit SVGPathElement(Document&); explicit SVGPathElement(Document&);
Path AsPath() const override; Path AsPath() const override;
......
...@@ -97,8 +97,6 @@ void SVGPatternElement::Trace(blink::Visitor* visitor) { ...@@ -97,8 +97,6 @@ void SVGPatternElement::Trace(blink::Visitor* visitor) {
SVGFitToViewBox::Trace(visitor); SVGFitToViewBox::Trace(visitor);
} }
DEFINE_NODE_FACTORY(SVGPatternElement)
void SVGPatternElement::BuildPendingResource() { void SVGPatternElement::BuildPendingResource() {
ClearResourceReferences(); ClearResourceReferences();
if (!isConnected()) if (!isConnected())
......
...@@ -45,8 +45,6 @@ class SVGPatternElement final : public SVGElement, ...@@ -45,8 +45,6 @@ class SVGPatternElement final : public SVGElement,
USING_GARBAGE_COLLECTED_MIXIN(SVGPatternElement); USING_GARBAGE_COLLECTED_MIXIN(SVGPatternElement);
public: public:
DECLARE_NODE_FACTORY(SVGPatternElement);
explicit SVGPatternElement(Document&); explicit SVGPatternElement(Document&);
void CollectPatternAttributes(PatternAttributes&) const; void CollectPatternAttributes(PatternAttributes&) const;
......
...@@ -44,12 +44,6 @@ class SVGPoint final : public SVGPropertyHelper<SVGPoint> { ...@@ -44,12 +44,6 @@ class SVGPoint final : public SVGPropertyHelper<SVGPoint> {
public: public:
typedef SVGPointTearOff TearOffType; typedef SVGPointTearOff TearOffType;
static SVGPoint* Create() { return MakeGarbageCollected<SVGPoint>(); }
static SVGPoint* Create(const FloatPoint& point) {
return MakeGarbageCollected<SVGPoint>(point);
}
SVGPoint(); SVGPoint();
explicit SVGPoint(const FloatPoint&); explicit SVGPoint(const FloatPoint&);
......
...@@ -44,8 +44,6 @@ class SVGPointList final ...@@ -44,8 +44,6 @@ class SVGPointList final
public: public:
typedef SVGPointListTearOff TearOffType; typedef SVGPointListTearOff TearOffType;
static SVGPointList* Create() { return MakeGarbageCollected<SVGPointList>(); }
SVGPointList(); SVGPointList();
~SVGPointList() override; ~SVGPointList() override;
......
...@@ -41,14 +41,6 @@ class SVGPointListTearOff final ...@@ -41,14 +41,6 @@ class SVGPointListTearOff final
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
static SVGPointListTearOff* Create(
SVGPointList* target,
SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) {
return MakeGarbageCollected<SVGPointListTearOff>(target, binding,
property_is_anim_val);
}
SVGPointListTearOff(SVGPointList* target, SVGPointListTearOff(SVGPointList* target,
SVGAnimatedPropertyBase* binding, SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) PropertyIsAnimValType property_is_anim_val)
......
...@@ -42,16 +42,6 @@ class SVGPointTearOff : public SVGPropertyTearOff<SVGPoint> { ...@@ -42,16 +42,6 @@ class SVGPointTearOff : public SVGPropertyTearOff<SVGPoint> {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
static SVGPointTearOff* Create(SVGPoint* target,
SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) {
return MakeGarbageCollected<SVGPointTearOff>(target, binding,
property_is_anim_val);
}
static SVGPointTearOff* Create(SVGPoint* target,
SVGElement* context_element) {
return MakeGarbageCollected<SVGPointTearOff>(target, context_element);
}
static SVGPointTearOff* CreateDetached(const FloatPoint&); static SVGPointTearOff* CreateDetached(const FloatPoint&);
SVGPointTearOff(SVGPoint*, SVGPointTearOff(SVGPoint*,
......
...@@ -27,8 +27,6 @@ namespace blink { ...@@ -27,8 +27,6 @@ namespace blink {
SVGPolygonElement::SVGPolygonElement(Document& document) SVGPolygonElement::SVGPolygonElement(Document& document)
: SVGPolyElement(svg_names::kPolygonTag, document) {} : SVGPolyElement(svg_names::kPolygonTag, document) {}
DEFINE_NODE_FACTORY(SVGPolygonElement)
Path SVGPolygonElement::AsPath() const { Path SVGPolygonElement::AsPath() const {
Path path = AsPathFromPoints(); Path path = AsPathFromPoints();
path.CloseSubpath(); path.CloseSubpath();
......
...@@ -29,8 +29,6 @@ class SVGPolygonElement final : public SVGPolyElement { ...@@ -29,8 +29,6 @@ class SVGPolygonElement final : public SVGPolyElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGPolygonElement);
explicit SVGPolygonElement(Document&); explicit SVGPolygonElement(Document&);
Path AsPath() const override; Path AsPath() const override;
......
...@@ -27,8 +27,6 @@ namespace blink { ...@@ -27,8 +27,6 @@ namespace blink {
SVGPolylineElement::SVGPolylineElement(Document& document) SVGPolylineElement::SVGPolylineElement(Document& document)
: SVGPolyElement(svg_names::kPolylineTag, document) {} : SVGPolyElement(svg_names::kPolylineTag, document) {}
DEFINE_NODE_FACTORY(SVGPolylineElement)
Path SVGPolylineElement::AsPath() const { Path SVGPolylineElement::AsPath() const {
return AsPathFromPoints(); return AsPathFromPoints();
} }
......
...@@ -29,8 +29,6 @@ class SVGPolylineElement final : public SVGPolyElement { ...@@ -29,8 +29,6 @@ class SVGPolylineElement final : public SVGPolyElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGPolylineElement);
explicit SVGPolylineElement(Document&); explicit SVGPolylineElement(Document&);
Path AsPath() const override; Path AsPath() const override;
......
...@@ -55,10 +55,6 @@ class SVGPreserveAspectRatio final ...@@ -55,10 +55,6 @@ class SVGPreserveAspectRatio final
typedef SVGPreserveAspectRatioTearOff TearOffType; typedef SVGPreserveAspectRatioTearOff TearOffType;
static SVGPreserveAspectRatio* Create() {
return MakeGarbageCollected<SVGPreserveAspectRatio>();
}
SVGPreserveAspectRatio(); SVGPreserveAspectRatio();
virtual SVGPreserveAspectRatio* Clone() const; virtual SVGPreserveAspectRatio* Clone() const;
......
...@@ -73,14 +73,6 @@ class SVGPreserveAspectRatioTearOff final ...@@ -73,14 +73,6 @@ class SVGPreserveAspectRatioTearOff final
kSvgMeetorsliceSlice = SVGPreserveAspectRatio::kSvgMeetorsliceSlice kSvgMeetorsliceSlice = SVGPreserveAspectRatio::kSvgMeetorsliceSlice
}; };
static SVGPreserveAspectRatioTearOff* Create(
SVGPreserveAspectRatio* target,
SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) {
return MakeGarbageCollected<SVGPreserveAspectRatioTearOff>(
target, binding, property_is_anim_val);
}
SVGPreserveAspectRatioTearOff(SVGPreserveAspectRatio*, SVGPreserveAspectRatioTearOff(SVGPreserveAspectRatio*,
SVGAnimatedPropertyBase* binding, SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType); PropertyIsAnimValType);
......
...@@ -83,8 +83,6 @@ void SVGRadialGradientElement::Trace(blink::Visitor* visitor) { ...@@ -83,8 +83,6 @@ void SVGRadialGradientElement::Trace(blink::Visitor* visitor) {
SVGGradientElement::Trace(visitor); SVGGradientElement::Trace(visitor);
} }
DEFINE_NODE_FACTORY(SVGRadialGradientElement)
void SVGRadialGradientElement::SvgAttributeChanged( void SVGRadialGradientElement::SvgAttributeChanged(
const QualifiedName& attr_name) { const QualifiedName& attr_name) {
if (attr_name == svg_names::kCxAttr || attr_name == svg_names::kCyAttr || if (attr_name == svg_names::kCxAttr || attr_name == svg_names::kCyAttr ||
......
...@@ -33,8 +33,6 @@ class SVGRadialGradientElement final : public SVGGradientElement { ...@@ -33,8 +33,6 @@ class SVGRadialGradientElement final : public SVGGradientElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGRadialGradientElement);
explicit SVGRadialGradientElement(Document&); explicit SVGRadialGradientElement(Document&);
bool CollectGradientAttributes(RadialGradientAttributes&); bool CollectGradientAttributes(RadialGradientAttributes&);
......
...@@ -33,18 +33,12 @@ class SVGRect final : public SVGPropertyHelper<SVGRect> { ...@@ -33,18 +33,12 @@ class SVGRect final : public SVGPropertyHelper<SVGRect> {
public: public:
typedef SVGRectTearOff TearOffType; typedef SVGRectTearOff TearOffType;
static SVGRect* Create() { return MakeGarbageCollected<SVGRect>(); }
static SVGRect* CreateInvalid() { static SVGRect* CreateInvalid() {
SVGRect* rect = MakeGarbageCollected<SVGRect>(); SVGRect* rect = MakeGarbageCollected<SVGRect>();
rect->SetInvalid(); rect->SetInvalid();
return rect; return rect;
} }
static SVGRect* Create(const FloatRect& rect) {
return MakeGarbageCollected<SVGRect>(rect);
}
SVGRect(); SVGRect();
SVGRect(const FloatRect&); SVGRect(const FloatRect&);
......
...@@ -82,8 +82,6 @@ void SVGRectElement::Trace(blink::Visitor* visitor) { ...@@ -82,8 +82,6 @@ void SVGRectElement::Trace(blink::Visitor* visitor) {
SVGGeometryElement::Trace(visitor); SVGGeometryElement::Trace(visitor);
} }
DEFINE_NODE_FACTORY(SVGRectElement)
Path SVGRectElement::AsPath() const { Path SVGRectElement::AsPath() const {
Path path; Path path;
......
...@@ -31,8 +31,6 @@ class SVGRectElement final : public SVGGeometryElement { ...@@ -31,8 +31,6 @@ class SVGRectElement final : public SVGGeometryElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGRectElement);
explicit SVGRectElement(Document&); explicit SVGRectElement(Document&);
Path AsPath() const override; Path AsPath() const override;
......
...@@ -40,12 +40,6 @@ class SVGRectTearOff : public SVGPropertyTearOff<SVGRect> { ...@@ -40,12 +40,6 @@ class SVGRectTearOff : public SVGPropertyTearOff<SVGRect> {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
static SVGRectTearOff* Create(SVGRect* target,
SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) {
return MakeGarbageCollected<SVGRectTearOff>(target, binding,
property_is_anim_val);
}
static SVGRectTearOff* CreateDetached(const FloatRect&); static SVGRectTearOff* CreateDetached(const FloatRect&);
SVGRectTearOff(SVGRect*, SVGRectTearOff(SVGRect*,
......
...@@ -40,11 +40,6 @@ SVGScriptElement::SVGScriptElement(Document& document, ...@@ -40,11 +40,6 @@ SVGScriptElement::SVGScriptElement(Document& document,
loader_(InitializeScriptLoader(flags.IsCreatedByParser(), loader_(InitializeScriptLoader(flags.IsCreatedByParser(),
flags.WasAlreadyStarted())) {} flags.WasAlreadyStarted())) {}
SVGScriptElement* SVGScriptElement::Create(Document& document,
const CreateElementFlags flags) {
return MakeGarbageCollected<SVGScriptElement>(document, flags);
}
void SVGScriptElement::ParseAttribute( void SVGScriptElement::ParseAttribute(
const AttributeModificationParams& params) { const AttributeModificationParams& params) {
if (params.name == html_names::kOnerrorAttr) { if (params.name == html_names::kOnerrorAttr) {
......
...@@ -39,8 +39,6 @@ class SVGScriptElement final : public SVGElement, ...@@ -39,8 +39,6 @@ class SVGScriptElement final : public SVGElement,
USING_GARBAGE_COLLECTED_MIXIN(SVGScriptElement); USING_GARBAGE_COLLECTED_MIXIN(SVGScriptElement);
public: public:
static SVGScriptElement* Create(Document&, const CreateElementFlags);
SVGScriptElement(Document&, const CreateElementFlags); SVGScriptElement(Document&, const CreateElementFlags);
ScriptLoader* Loader() const final { return loader_.Get(); } ScriptLoader* Loader() const final { return loader_.Get(); }
......
...@@ -29,8 +29,6 @@ SVGSetElement::SVGSetElement(Document& document) ...@@ -29,8 +29,6 @@ SVGSetElement::SVGSetElement(Document& document)
SetAnimationMode(kToAnimation); SetAnimationMode(kToAnimation);
} }
DEFINE_NODE_FACTORY(SVGSetElement)
void SVGSetElement::UpdateAnimationMode() { void SVGSetElement::UpdateAnimationMode() {
// No-op, as <set> has a constant animation mode of ToAnimation. // No-op, as <set> has a constant animation mode of ToAnimation.
// See: http://www.w3.org/TR/SVG/single-page.html#animate-SetElement // See: http://www.w3.org/TR/SVG/single-page.html#animate-SetElement
......
...@@ -31,8 +31,6 @@ class CORE_EXPORT SVGSetElement final : public SVGAnimateElement { ...@@ -31,8 +31,6 @@ class CORE_EXPORT SVGSetElement final : public SVGAnimateElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGSetElement);
explicit SVGSetElement(Document&); explicit SVGSetElement(Document&);
private: private:
......
...@@ -45,8 +45,6 @@ void SVGStopElement::Trace(blink::Visitor* visitor) { ...@@ -45,8 +45,6 @@ void SVGStopElement::Trace(blink::Visitor* visitor) {
SVGElement::Trace(visitor); SVGElement::Trace(visitor);
} }
DEFINE_NODE_FACTORY(SVGStopElement)
namespace { namespace {
void InvalidateInstancesAndAncestorResources(SVGStopElement* stop_element) { void InvalidateInstancesAndAncestorResources(SVGStopElement* stop_element) {
......
...@@ -33,8 +33,6 @@ class SVGStopElement final : public SVGElement { ...@@ -33,8 +33,6 @@ class SVGStopElement final : public SVGElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGStopElement);
explicit SVGStopElement(Document&); explicit SVGStopElement(Document&);
Color StopColorIncludingOpacity() const; Color StopColorIncludingOpacity() const;
......
...@@ -43,12 +43,6 @@ class SVGString final : public SVGPropertyBase { ...@@ -43,12 +43,6 @@ class SVGString final : public SVGPropertyBase {
typedef void TearOffType; typedef void TearOffType;
typedef String PrimitiveType; typedef String PrimitiveType;
static SVGString* Create() { return MakeGarbageCollected<SVGString>(); }
static SVGString* Create(const String& value) {
return MakeGarbageCollected<SVGString>(value);
}
SVGString() = default; SVGString() = default;
explicit SVGString(const String& value) : value_(value) {} explicit SVGString(const String& value) : value_(value) {}
......
...@@ -112,10 +112,6 @@ class SVGStringListBase : public SVGPropertyBase { ...@@ -112,10 +112,6 @@ class SVGStringListBase : public SVGPropertyBase {
template <char list_delimiter> template <char list_delimiter>
class SVGStringList final : public SVGStringListBase { class SVGStringList final : public SVGStringListBase {
public: public:
static SVGStringList<list_delimiter>* Create() {
return MakeGarbageCollected<SVGStringList<list_delimiter>>();
}
SVGStringList() = default; SVGStringList() = default;
~SVGStringList() override = default; ~SVGStringList() override = default;
......
...@@ -40,14 +40,6 @@ class SVGStringListTearOff : public SVGPropertyTearOff<SVGStringListBase> { ...@@ -40,14 +40,6 @@ class SVGStringListTearOff : public SVGPropertyTearOff<SVGStringListBase> {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
static SVGStringListTearOff* Create(
SVGStringListBase* target,
SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) {
return MakeGarbageCollected<SVGStringListTearOff>(target, binding,
property_is_anim_val);
}
SVGStringListTearOff(SVGStringListBase*, SVGStringListTearOff(SVGStringListBase*,
SVGAnimatedPropertyBase* binding, SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType); PropertyIsAnimValType);
......
...@@ -38,11 +38,6 @@ SVGStyleElement::SVGStyleElement(Document& document, ...@@ -38,11 +38,6 @@ SVGStyleElement::SVGStyleElement(Document& document,
SVGStyleElement::~SVGStyleElement() = default; SVGStyleElement::~SVGStyleElement() = default;
SVGStyleElement* SVGStyleElement::Create(Document& document,
const CreateElementFlags flags) {
return MakeGarbageCollected<SVGStyleElement>(document, flags);
}
bool SVGStyleElement::disabled() const { bool SVGStyleElement::disabled() const {
if (!sheet_) if (!sheet_)
return false; return false;
......
...@@ -32,8 +32,6 @@ class SVGStyleElement final : public SVGElement, public StyleElement { ...@@ -32,8 +32,6 @@ class SVGStyleElement final : public SVGElement, public StyleElement {
USING_GARBAGE_COLLECTED_MIXIN(SVGStyleElement); USING_GARBAGE_COLLECTED_MIXIN(SVGStyleElement);
public: public:
static SVGStyleElement* Create(Document&, const CreateElementFlags);
SVGStyleElement(Document&, const CreateElementFlags); SVGStyleElement(Document&, const CreateElementFlags);
~SVGStyleElement() override; ~SVGStyleElement() override;
......
...@@ -100,8 +100,6 @@ SVGSVGElement::SVGSVGElement(Document& doc) ...@@ -100,8 +100,6 @@ SVGSVGElement::SVGSVGElement(Document& doc)
UseCounter::Count(doc, WebFeature::kSVGSVGElement); UseCounter::Count(doc, WebFeature::kSVGSVGElement);
} }
DEFINE_NODE_FACTORY(SVGSVGElement)
SVGSVGElement::~SVGSVGElement() = default; SVGSVGElement::~SVGSVGElement() = default;
float SVGSVGElement::currentScale() const { float SVGSVGElement::currentScale() const {
...@@ -122,10 +120,6 @@ void SVGSVGElement::setCurrentScale(float scale) { ...@@ -122,10 +120,6 @@ void SVGSVGElement::setCurrentScale(float scale) {
class SVGCurrentTranslateTearOff : public SVGPointTearOff { class SVGCurrentTranslateTearOff : public SVGPointTearOff {
public: public:
static SVGCurrentTranslateTearOff* Create(SVGSVGElement* context_element) {
return MakeGarbageCollected<SVGCurrentTranslateTearOff>(context_element);
}
SVGCurrentTranslateTearOff(SVGSVGElement* context_element) SVGCurrentTranslateTearOff(SVGSVGElement* context_element)
: SVGPointTearOff(context_element->translation_, context_element) {} : SVGPointTearOff(context_element->translation_, context_element) {}
......
...@@ -47,8 +47,6 @@ class SVGSVGElement final : public SVGGraphicsElement, ...@@ -47,8 +47,6 @@ class SVGSVGElement final : public SVGGraphicsElement,
USING_GARBAGE_COLLECTED_MIXIN(SVGSVGElement); USING_GARBAGE_COLLECTED_MIXIN(SVGSVGElement);
public: public:
DECLARE_NODE_FACTORY(SVGSVGElement);
explicit SVGSVGElement(Document&); explicit SVGSVGElement(Document&);
float IntrinsicWidth() const; float IntrinsicWidth() const;
......
...@@ -31,8 +31,6 @@ SVGSwitchElement::SVGSwitchElement(Document& document) ...@@ -31,8 +31,6 @@ SVGSwitchElement::SVGSwitchElement(Document& document)
UseCounter::Count(document, WebFeature::kSVGSwitchElement); UseCounter::Count(document, WebFeature::kSVGSwitchElement);
} }
DEFINE_NODE_FACTORY(SVGSwitchElement)
LayoutObject* SVGSwitchElement::CreateLayoutObject(const ComputedStyle&, LayoutObject* SVGSwitchElement::CreateLayoutObject(const ComputedStyle&,
LegacyLayout) { LegacyLayout) {
return new LayoutSVGTransformableContainer(this); return new LayoutSVGTransformableContainer(this);
......
...@@ -29,8 +29,6 @@ class SVGSwitchElement final : public SVGGraphicsElement { ...@@ -29,8 +29,6 @@ class SVGSwitchElement final : public SVGGraphicsElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGSwitchElement);
explicit SVGSwitchElement(Document&); explicit SVGSwitchElement(Document&);
private: private:
......
...@@ -33,8 +33,6 @@ void SVGSymbolElement::Trace(blink::Visitor* visitor) { ...@@ -33,8 +33,6 @@ void SVGSymbolElement::Trace(blink::Visitor* visitor) {
SVGFitToViewBox::Trace(visitor); SVGFitToViewBox::Trace(visitor);
} }
DEFINE_NODE_FACTORY(SVGSymbolElement)
void SVGSymbolElement::SvgAttributeChanged(const QualifiedName& attr_name) { void SVGSymbolElement::SvgAttributeChanged(const QualifiedName& attr_name) {
if (SVGFitToViewBox::IsKnownAttribute(attr_name)) if (SVGFitToViewBox::IsKnownAttribute(attr_name))
InvalidateInstances(); InvalidateInstances();
......
...@@ -31,8 +31,6 @@ class SVGSymbolElement final : public SVGElement, public SVGFitToViewBox { ...@@ -31,8 +31,6 @@ class SVGSymbolElement final : public SVGElement, public SVGFitToViewBox {
USING_GARBAGE_COLLECTED_MIXIN(SVGSymbolElement); USING_GARBAGE_COLLECTED_MIXIN(SVGSymbolElement);
public: public:
DECLARE_NODE_FACTORY(SVGSymbolElement);
explicit SVGSymbolElement(Document&); explicit SVGSymbolElement(Document&);
void Trace(blink::Visitor*) override; void Trace(blink::Visitor*) override;
......
...@@ -27,8 +27,6 @@ namespace blink { ...@@ -27,8 +27,6 @@ namespace blink {
SVGTextElement::SVGTextElement(Document& doc) SVGTextElement::SVGTextElement(Document& doc)
: SVGTextPositioningElement(svg_names::kTextTag, doc) {} : SVGTextPositioningElement(svg_names::kTextTag, doc) {}
DEFINE_NODE_FACTORY(SVGTextElement)
LayoutObject* SVGTextElement::CreateLayoutObject(const ComputedStyle&, LayoutObject* SVGTextElement::CreateLayoutObject(const ComputedStyle&,
LegacyLayout) { LegacyLayout) {
return new LayoutSVGText(this); return new LayoutSVGText(this);
......
...@@ -29,8 +29,6 @@ class SVGTextElement final : public SVGTextPositioningElement { ...@@ -29,8 +29,6 @@ class SVGTextElement final : public SVGTextPositioningElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGTextElement);
explicit SVGTextElement(Document&); explicit SVGTextElement(Document&);
private: private:
......
...@@ -69,8 +69,6 @@ SVGTextPathElement::SVGTextPathElement(Document& document) ...@@ -69,8 +69,6 @@ SVGTextPathElement::SVGTextPathElement(Document& document)
AddToPropertyMap(spacing_); AddToPropertyMap(spacing_);
} }
DEFINE_NODE_FACTORY(SVGTextPathElement)
SVGTextPathElement::~SVGTextPathElement() = default; SVGTextPathElement::~SVGTextPathElement() = default;
void SVGTextPathElement::Trace(blink::Visitor* visitor) { void SVGTextPathElement::Trace(blink::Visitor* visitor) {
......
...@@ -56,8 +56,6 @@ class SVGTextPathElement final : public SVGTextContentElement, ...@@ -56,8 +56,6 @@ class SVGTextPathElement final : public SVGTextContentElement,
kTextpathSpacingtypeExact = kSVGTextPathSpacingExact kTextpathSpacingtypeExact = kSVGTextPathSpacingExact
}; };
DECLARE_NODE_FACTORY(SVGTextPathElement);
explicit SVGTextPathElement(Document&); explicit SVGTextPathElement(Document&);
SVGAnimatedLength* startOffset() const { return start_offset_.Get(); } SVGAnimatedLength* startOffset() const { return start_offset_.Get(); }
......
...@@ -32,8 +32,6 @@ SVGTitleElement::SVGTitleElement(Document& document) ...@@ -32,8 +32,6 @@ SVGTitleElement::SVGTitleElement(Document& document)
: SVGElement(svg_names::kTitleTag, document), : SVGElement(svg_names::kTitleTag, document),
ignore_title_updates_when_children_change_(false) {} ignore_title_updates_when_children_change_(false) {}
DEFINE_NODE_FACTORY(SVGTitleElement)
Node::InsertionNotificationRequest SVGTitleElement::InsertedInto( Node::InsertionNotificationRequest SVGTitleElement::InsertedInto(
ContainerNode& root_parent) { ContainerNode& root_parent) {
SVGElement::InsertedInto(root_parent); SVGElement::InsertedInto(root_parent);
......
...@@ -29,8 +29,6 @@ class SVGTitleElement final : public SVGElement { ...@@ -29,8 +29,6 @@ class SVGTitleElement final : public SVGElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGTitleElement);
explicit SVGTitleElement(Document&); explicit SVGTitleElement(Document&);
void SetText(const String&); void SetText(const String&);
......
...@@ -50,18 +50,6 @@ class SVGTransform final : public SVGPropertyBase { ...@@ -50,18 +50,6 @@ class SVGTransform final : public SVGPropertyBase {
kConstructZeroTransform kConstructZeroTransform
}; };
static SVGTransform* Create() { return MakeGarbageCollected<SVGTransform>(); }
static SVGTransform* Create(
SVGTransformType type,
ConstructionMode mode = kConstructIdentityTransform) {
return MakeGarbageCollected<SVGTransform>(type, mode);
}
static SVGTransform* Create(const AffineTransform& affine_transform) {
return MakeGarbageCollected<SVGTransform>(affine_transform);
}
SVGTransform(); SVGTransform();
explicit SVGTransform(SVGTransformType, explicit SVGTransform(SVGTransformType,
ConstructionMode = kConstructIdentityTransform); ConstructionMode = kConstructIdentityTransform);
......
...@@ -45,14 +45,6 @@ class SVGTransformListTearOff final ...@@ -45,14 +45,6 @@ class SVGTransformListTearOff final
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
static SVGTransformListTearOff* Create(
SVGTransformList* target,
SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) {
return MakeGarbageCollected<SVGTransformListTearOff>(target, binding,
property_is_anim_val);
}
SVGTransformListTearOff(SVGTransformList*, SVGTransformListTearOff(SVGTransformList*,
SVGAnimatedPropertyBase* binding, SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType); PropertyIsAnimValType);
......
...@@ -59,11 +59,6 @@ SVGTransformTearOff* SVGTransformTearOff::CreateDetached() { ...@@ -59,11 +59,6 @@ SVGTransformTearOff* SVGTransformTearOff::CreateDetached() {
nullptr, kPropertyIsNotAnimVal); nullptr, kPropertyIsNotAnimVal);
} }
SVGTransformTearOff* SVGTransformTearOff::Create(SVGMatrixTearOff* matrix) {
return Create(SVGTransform::Create(matrix->Value()), nullptr,
kPropertyIsNotAnimVal);
}
SVGMatrixTearOff* SVGTransformTearOff::matrix() { SVGMatrixTearOff* SVGTransformTearOff::matrix() {
if (!matrix_tearoff_) if (!matrix_tearoff_)
matrix_tearoff_ = MakeGarbageCollected<SVGMatrixTearOff>(this); matrix_tearoff_ = MakeGarbageCollected<SVGMatrixTearOff>(this);
......
...@@ -53,15 +53,7 @@ class SVGTransformTearOff final : public SVGPropertyTearOff<SVGTransform> { ...@@ -53,15 +53,7 @@ class SVGTransformTearOff final : public SVGPropertyTearOff<SVGTransform> {
kSvgTransformSkewy = static_cast<int>(blink::SVGTransformType::kSkewy), kSvgTransformSkewy = static_cast<int>(blink::SVGTransformType::kSkewy),
}; };
static SVGTransformTearOff* Create(
SVGTransform* target,
SVGAnimatedPropertyBase* binding,
PropertyIsAnimValType property_is_anim_val) {
return MakeGarbageCollected<SVGTransformTearOff>(target, binding,
property_is_anim_val);
}
static SVGTransformTearOff* CreateDetached(); static SVGTransformTearOff* CreateDetached();
static SVGTransformTearOff* Create(SVGMatrixTearOff*);
SVGTransformTearOff(SVGMatrixTearOff*); SVGTransformTearOff(SVGMatrixTearOff*);
SVGTransformTearOff(SVGTransform*, SVGTransformTearOff(SVGTransform*,
......
...@@ -28,8 +28,6 @@ namespace blink { ...@@ -28,8 +28,6 @@ namespace blink {
SVGTSpanElement::SVGTSpanElement(Document& document) SVGTSpanElement::SVGTSpanElement(Document& document)
: SVGTextPositioningElement(svg_names::kTSpanTag, document) {} : SVGTextPositioningElement(svg_names::kTSpanTag, document) {}
DEFINE_NODE_FACTORY(SVGTSpanElement)
LayoutObject* SVGTSpanElement::CreateLayoutObject(const ComputedStyle&, LayoutObject* SVGTSpanElement::CreateLayoutObject(const ComputedStyle&,
LegacyLayout) { LegacyLayout) {
return new LayoutSVGTSpan(this); return new LayoutSVGTSpan(this);
......
...@@ -29,8 +29,6 @@ class SVGTSpanElement final : public SVGTextPositioningElement { ...@@ -29,8 +29,6 @@ class SVGTSpanElement final : public SVGTextPositioningElement {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
DECLARE_NODE_FACTORY(SVGTSpanElement);
explicit SVGTSpanElement(Document&); explicit SVGTSpanElement(Document&);
private: private:
......
...@@ -36,6 +36,4 @@ SVGUnknownElement::SVGUnknownElement(const QualifiedName& tag_name, ...@@ -36,6 +36,4 @@ SVGUnknownElement::SVGUnknownElement(const QualifiedName& tag_name,
Document& document) Document& document)
: SVGElement(tag_name, document) {} : SVGElement(tag_name, document) {}
DEFINE_ELEMENT_FACTORY_WITH_TAGNAME(SVGUnknownElement)
} // namespace blink } // namespace blink
...@@ -45,7 +45,6 @@ namespace blink { ...@@ -45,7 +45,6 @@ namespace blink {
// layout such elements. // layout such elements.
class SVGUnknownElement final : public SVGElement { class SVGUnknownElement final : public SVGElement {
public: public:
DECLARE_ELEMENT_FACTORY_WITH_TAGNAME(SVGUnknownElement);
SVGUnknownElement(const QualifiedName&, Document&); SVGUnknownElement(const QualifiedName&, Document&);
......
...@@ -30,8 +30,6 @@ SVGViewElement::SVGViewElement(Document& document) ...@@ -30,8 +30,6 @@ SVGViewElement::SVGViewElement(Document& document)
UseCounter::Count(document, WebFeature::kSVGViewElement); UseCounter::Count(document, WebFeature::kSVGViewElement);
} }
DEFINE_NODE_FACTORY(SVGViewElement)
void SVGViewElement::Trace(blink::Visitor* visitor) { void SVGViewElement::Trace(blink::Visitor* visitor) {
SVGElement::Trace(visitor); SVGElement::Trace(visitor);
SVGFitToViewBox::Trace(visitor); SVGFitToViewBox::Trace(visitor);
......
...@@ -35,8 +35,6 @@ class SVGViewElement final : public SVGElement, ...@@ -35,8 +35,6 @@ class SVGViewElement final : public SVGElement,
USING_GARBAGE_COLLECTED_MIXIN(SVGViewElement); USING_GARBAGE_COLLECTED_MIXIN(SVGViewElement);
public: public:
DECLARE_NODE_FACTORY(SVGViewElement);
explicit SVGViewElement(Document&); explicit SVGViewElement(Document&);
void Trace(blink::Visitor*) override; void Trace(blink::Visitor*) override;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment