Bug 1035394 - Fix dangerous public destructors in content/svg - r=heycam

content/svg/content/src/SVGAElement.cpp

@@ -68,6 +68,10 @@ SVGAElement::SVGAElement(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo)
 {
 }
 
+SVGAElement::~SVGAElement()
+{
+}
+
 already_AddRefed<SVGAnimatedString>
 SVGAElement::Href()
 {

content/svg/content/src/SVGAElement.h

@@ -71,6 +71,7 @@ public:
   void SetDownload(const nsAString & aDownload, ErrorResult& rv);
 
 protected:
+  virtual ~SVGAElement();
 
   virtual StringAttributesInfo GetStringInfo() MOZ_OVERRIDE;
 

content/svg/content/src/SVGAnimationElement.cpp

@@ -49,6 +49,10 @@ SVGAnimationElement::SVGAnimationElement(already_AddRefed<mozilla::dom::NodeInfo
 {
 }
 
+SVGAnimationElement::~SVGAnimationElement()
+{
+}
+
 nsresult
 SVGAnimationElement::Init()
 {

content/svg/content/src/SVGAnimationElement.h

@@ -29,6 +29,7 @@ class SVGAnimationElement : public SVGAnimationElementBase,
 protected:
   SVGAnimationElement(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo);
   nsresult Init();
+  virtual ~SVGAnimationElement();
 
 public:
   // interfaces:

content/svg/content/src/SVGComponentTransferFunctionElement.h

@@ -32,6 +32,8 @@ protected:
   {
   }
 
+  virtual ~SVGComponentTransferFunctionElement() {}
+
 public:
   typedef gfx::AttributeMap AttributeMap;
 

content/svg/content/src/SVGIFrameElement.cpp

@@ -53,6 +53,10 @@ SVGIFrameElement::SVGIFrameElement(already_AddRefed<mozilla::dom::NodeInfo>& aNo
 {
 }
 
+SVGIFrameElement::~SVGIFrameElement()
+{
+}
+
 //----------------------------------------------------------------------
 // nsSVGElement methods
 

content/svg/content/src/SVGIFrameElement.h

@@ -43,6 +43,8 @@ class SVGIFrameElement MOZ_FINAL : public SVGIFrameElementBase,
                    mozilla::dom::FromParser aFromParser);
   virtual JSObject* WrapNode(JSContext *aCx) MOZ_OVERRIDE;
 
+  ~SVGIFrameElement();
+
 public:
   // interface
   NS_DECL_ISUPPORTS_INHERITED

content/svg/content/src/SVGSVGElement.h

@@ -67,6 +67,9 @@ public:
   virtual nsISupports* GetParentObject() MOZ_OVERRIDE;
 
   nsRefPtr<SVGSVGElement> mElement;
+
+private:
+  ~DOMSVGTranslatePoint() {}
 };
 
 class svgFloatSize {

content/svg/content/src/SVGScriptElement.cpp

@@ -46,6 +46,10 @@ SVGScriptElement::SVGScriptElement(already_AddRefed<mozilla::dom::NodeInfo>& aNo
   AddMutationObserver(this);
 }
 
+SVGScriptElement::~SVGScriptElement()
+{
+}
+
 //----------------------------------------------------------------------
 // nsIDOMNode methods
 

content/svg/content/src/SVGScriptElement.h

@@ -71,6 +71,8 @@ public:
   already_AddRefed<SVGAnimatedString> Href();
 
 protected:
+  ~SVGScriptElement();
+
   virtual StringAttributesInfo GetStringInfo() MOZ_OVERRIDE;
 
   enum { HREF };

content/svg/content/src/SVGStyleElement.cpp

@@ -52,6 +52,9 @@ SVGStyleElement::SVGStyleElement(already_AddRefed<mozilla::dom::NodeInfo>& aNode
   AddMutationObserver(this);
 }
 
+SVGStyleElement::~SVGStyleElement()
+{
+}
 
 //----------------------------------------------------------------------
 // nsIDOMNode methods

content/svg/content/src/SVGStyleElement.h

@@ -27,6 +27,7 @@ protected:
   friend nsresult (::NS_NewSVGStyleElement(nsIContent **aResult,
                                            already_AddRefed<mozilla::dom::NodeInfo>&& aNodeInfo));
   SVGStyleElement(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo);
+  ~SVGStyleElement();
 
   virtual JSObject* WrapNode(JSContext *aCx) MOZ_OVERRIDE;
 

content/svg/content/src/SVGSwitchElement.cpp

@@ -43,6 +43,10 @@ SVGSwitchElement::SVGSwitchElement(already_AddRefed<mozilla::dom::NodeInfo>& aNo
 {
 }
 
+SVGSwitchElement::~SVGSwitchElement()
+{
+}
+
 void
 SVGSwitchElement::MaybeInvalidate()
 {

content/svg/content/src/SVGSwitchElement.h

@@ -25,6 +25,7 @@ protected:
   friend nsresult (::NS_NewSVGSwitchElement(nsIContent **aResult,
                                             already_AddRefed<mozilla::dom::NodeInfo>&& aNodeInfo));
   SVGSwitchElement(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo);
+  ~SVGSwitchElement();
   virtual JSObject* WrapNode(JSContext *aCx) MOZ_OVERRIDE;
 
 public:

content/svg/content/src/SVGSymbolElement.cpp

@@ -32,6 +32,9 @@ SVGSymbolElement::SVGSymbolElement(already_AddRefed<mozilla::dom::NodeInfo>& aNo
 {
 }
 
+SVGSymbolElement::~SVGSymbolElement()
+{
+}
 
 //----------------------------------------------------------------------
 // nsIDOMNode methods

content/svg/content/src/SVGSymbolElement.h

@@ -26,6 +26,7 @@ protected:
   friend nsresult (::NS_NewSVGSymbolElement(nsIContent **aResult,
                                             already_AddRefed<mozilla::dom::NodeInfo>&& aNodeInfo));
   SVGSymbolElement(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo);
+  ~SVGSymbolElement();
   virtual JSObject* WrapNode(JSContext *cx) MOZ_OVERRIDE;
 
 public:

content/svg/content/src/SVGTitleElement.cpp

@@ -34,6 +34,10 @@ SVGTitleElement::SVGTitleElement(already_AddRefed<mozilla::dom::NodeInfo>& aNode
   AddMutationObserver(this);
 }
 
+SVGTitleElement::~SVGTitleElement()
+{
+}
+
 void
 SVGTitleElement::CharacterDataChanged(nsIDocument *aDocument,
                                       nsIContent *aContent,

content/svg/content/src/SVGTitleElement.h

@@ -24,6 +24,7 @@ protected:
   friend nsresult (::NS_NewSVGTitleElement(nsIContent **aResult,
                                            already_AddRefed<mozilla::dom::NodeInfo>&& aNodeInfo));
   SVGTitleElement(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo);
+  ~SVGTitleElement();
 
   virtual JSObject* WrapNode(JSContext *aCx) MOZ_OVERRIDE;
 

content/svg/content/src/SVGZoomEvent.cpp

@@ -80,6 +80,9 @@ SVGZoomEvent::SVGZoomEvent(EventTarget* aOwner,
   }
 }
 
+SVGZoomEvent::~SVGZoomEvent()
+{
+}
 
 } // namespace dom
 } // namespace mozilla

content/svg/content/src/SVGZoomEvent.h

@@ -59,6 +59,8 @@ public:
   }
 
 private:
+  ~SVGZoomEvent();
+
   float mPreviousScale;
   float mNewScale;
   nsRefPtr<DOMSVGPoint> mPreviousTranslate;

content/svg/content/src/nsSVGFilters.h

@@ -52,6 +52,7 @@ protected:
 
   nsSVGFE(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo)
     : nsSVGFEBase(aNodeInfo) {}
+  virtual ~nsSVGFE() {}
 
 public:
   typedef mozilla::gfx::AttributeMap AttributeMap;
@@ -176,6 +177,8 @@ protected:
   nsSVGFELightingElement(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo)
     : nsSVGFELightingElementBase(aNodeInfo) {}
 
+  virtual ~nsSVGFELightingElement() {}
+
 public:
   // interfaces:
   NS_DECL_ISUPPORTS_INHERITED

content/svg/content/src/nsSVGPolyElement.cpp

@@ -29,6 +29,10 @@ nsSVGPolyElement::nsSVGPolyElement(already_AddRefed<mozilla::dom::NodeInfo>& aNo
 {
 }
 
+nsSVGPolyElement::~nsSVGPolyElement()
+{
+}
+
 already_AddRefed<DOMSVGPointList>
 nsSVGPolyElement::Points()
 {

content/svg/content/src/nsSVGPolyElement.h

@@ -23,6 +23,8 @@ class nsSVGPolyElement : public nsSVGPolyElementBase
 protected:
   nsSVGPolyElement(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo);
 
+  virtual ~nsSVGPolyElement();
+
 public:
   //interfaces