Bug 1542646 Part 2 - Use default keyword instead of empty constructors and destructors r=dholbert
authorlongsonr <longsonr@gmail.com>
Wed, 10 Apr 2019 05:34:35 +0100
changeset 468811 323ed046cecd35e2d82faa8c894da09cc5e4486c
parent 468810 fe062eac74d9d71fdb1fb3ebc8d75c2572f47c98
child 468812 0a048e81c20c98ff831765e43be0ac76e4870c49
push id35851
push userdvarga@mozilla.com
push dateWed, 10 Apr 2019 21:56:12 +0000
treeherdermozilla-central@30ca3c3abfe6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdholbert
bugs1542646
milestone68.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1542646 Part 2 - Use default keyword instead of empty constructors and destructors r=dholbert
dom/svg/SVGAnimatedLength.h
dom/svg/SVGAnimatedLengthList.h
dom/svg/SVGAnimatedPathSegList.h
dom/svg/SVGAnimatedPointList.h
dom/svg/SVGComponentTransferFunctionElement.h
dom/svg/SVGFilters.h
dom/svg/SVGIRect.h
dom/svg/SVGIntegerPairSMILType.h
dom/svg/SVGLengthList.h
dom/svg/SVGLengthListSMILType.h
dom/svg/SVGMatrix.h
dom/svg/SVGMotionSMILType.h
dom/svg/SVGNumberList.h
dom/svg/SVGNumberListSMILType.h
dom/svg/SVGNumberPairSMILType.h
dom/svg/SVGOrientSMILType.h
dom/svg/SVGPathData.h
dom/svg/SVGPathSegListSMILType.h
dom/svg/SVGPathSegUtils.h
dom/svg/SVGPointList.h
dom/svg/SVGPointListSMILType.h
dom/svg/SVGRect.h
dom/svg/SVGSVGElement.h
dom/svg/SVGStringList.h
dom/svg/SVGTests.h
dom/svg/SVGTransformList.h
dom/svg/SVGTransformListSMILType.h
dom/svg/SVGTransformableElement.h
dom/svg/SVGViewBoxSMILType.h
--- a/dom/svg/SVGAnimatedLength.h
+++ b/dom/svg/SVGAnimatedLength.h
@@ -28,17 +28,17 @@ class SMILValue;
 namespace dom {
 class DOMSVGAnimatedLength;
 class DOMSVGLength;
 class SVGAnimationElement;
 class SVGViewportElement;
 
 class UserSpaceMetrics {
  public:
-  virtual ~UserSpaceMetrics() {}
+  virtual ~UserSpaceMetrics() = default;
 
   virtual float GetEmLength() const = 0;
   virtual float GetExLength() const = 0;
   virtual float GetAxisLength(uint8_t aCtxType) const = 0;
 };
 
 class UserSpaceMetricsWithSize : public UserSpaceMetrics {
  public:
--- a/dom/svg/SVGAnimatedLengthList.h
+++ b/dom/svg/SVGAnimatedLengthList.h
@@ -37,17 +37,17 @@ class SVGElement;
  * consumers don't need to concern themselves with that.
  */
 class SVGAnimatedLengthList {
   // friends so that they can get write access to mBaseVal
   friend class dom::DOMSVGLength;
   friend class dom::DOMSVGLengthList;
 
  public:
-  SVGAnimatedLengthList() {}
+  SVGAnimatedLengthList() = default;
 
   /**
    * Because it's so important that mBaseVal and its DOMSVGLengthList wrapper
    * (if any) be kept in sync (see the comment in
    * DOMSVGAnimatedLengthList::InternalBaseValListWillChangeTo), this method
    * returns a const reference. Only our friend classes may get mutable
    * references to mBaseVal.
    */
--- a/dom/svg/SVGAnimatedPathSegList.h
+++ b/dom/svg/SVGAnimatedPathSegList.h
@@ -39,17 +39,17 @@ class SVGElement;
  * to know or worry about wrappers (or forget about them!) for the most part.
  */
 class SVGAnimatedPathSegList final {
   // friends so that they can get write access to mBaseVal and mAnimVal
   friend class DOMSVGPathSeg;
   friend class DOMSVGPathSegList;
 
  public:
-  SVGAnimatedPathSegList() {}
+  SVGAnimatedPathSegList() = default;
 
   /**
    * Because it's so important that mBaseVal and its DOMSVGPathSegList wrapper
    * (if any) be kept in sync (see the comment in
    * DOMSVGPathSegList::InternalListWillChangeTo), this method returns a const
    * reference. Only our friend classes may get mutable references to mBaseVal.
    */
   const SVGPathData& GetBaseValue() const { return mBaseVal; }
--- a/dom/svg/SVGAnimatedPointList.h
+++ b/dom/svg/SVGAnimatedPointList.h
@@ -38,17 +38,17 @@ class SVGElement;
  * to know or worry about wrappers (or forget about them!) for the most part.
  */
 class SVGAnimatedPointList {
   // friends so that they can get write access to mBaseVal and mAnimVal
   friend class DOMSVGPoint;
   friend class DOMSVGPointList;
 
  public:
-  SVGAnimatedPointList() {}
+  SVGAnimatedPointList() = default;
 
   /**
    * Because it's so important that mBaseVal and its DOMSVGPointList wrapper
    * (if any) be kept in sync (see the comment in
    * DOMSVGPointList::InternalListWillChangeTo), this method returns a const
    * reference. Only our friend classes may get mutable references to mBaseVal.
    */
   const SVGPointList& GetBaseValue() const { return mBaseVal; }
--- a/dom/svg/SVGComponentTransferFunctionElement.h
+++ b/dom/svg/SVGComponentTransferFunctionElement.h
@@ -29,17 +29,17 @@ typedef SVGFEUnstyledElement SVGComponen
 
 class SVGComponentTransferFunctionElement
     : public SVGComponentTransferFunctionElementBase {
  protected:
   explicit SVGComponentTransferFunctionElement(
       already_AddRefed<mozilla::dom::NodeInfo>&& aNodeInfo)
       : SVGComponentTransferFunctionElementBase(std::move(aNodeInfo)) {}
 
-  virtual ~SVGComponentTransferFunctionElement() {}
+  virtual ~SVGComponentTransferFunctionElement() = default;
 
  public:
   typedef gfx::ComponentTransferAttributes ComponentTransferAttributes;
 
   // interfaces:
   NS_DECLARE_STATIC_IID_ACCESSOR(
       NS_SVG_FE_COMPONENT_TRANSFER_FUNCTION_ELEMENT_CID)
 
--- a/dom/svg/SVGFilters.h
+++ b/dom/svg/SVGFilters.h
@@ -50,17 +50,17 @@ class SVGFE : public SVGFEBase {
   typedef mozilla::gfx::SourceSurface SourceSurface;
   typedef mozilla::gfx::Size Size;
   typedef mozilla::gfx::IntRect IntRect;
   typedef mozilla::gfx::ColorSpace ColorSpace;
   typedef mozilla::gfx::FilterPrimitiveDescription FilterPrimitiveDescription;
 
   explicit SVGFE(already_AddRefed<mozilla::dom::NodeInfo>&& aNodeInfo)
       : SVGFEBase(std::move(aNodeInfo)) {}
-  virtual ~SVGFE() {}
+  virtual ~SVGFE() = default;
 
  public:
   typedef mozilla::gfx::PrimitiveAttributes PrimitiveAttributes;
 
   ColorSpace GetInputColorSpace(int32_t aInputIndex,
                                 ColorSpace aUnchangedInputColorSpace) {
     return OperatesOnSRGB(aInputIndex,
                           aUnchangedInputColorSpace == ColorSpace::SRGB)
@@ -178,17 +178,17 @@ class SVGFEUnstyledElement : public SVGF
 typedef SVGFE SVGFELightingElementBase;
 
 class SVGFELightingElement : public SVGFELightingElementBase {
  protected:
   explicit SVGFELightingElement(
       already_AddRefed<mozilla::dom::NodeInfo>&& aNodeInfo)
       : SVGFELightingElementBase(std::move(aNodeInfo)) {}
 
-  virtual ~SVGFELightingElement() {}
+  virtual ~SVGFELightingElement() = default;
 
  public:
   // interfaces:
   NS_INLINE_DECL_REFCOUNTING_INHERITED(SVGFELightingElement,
                                        SVGFELightingElementBase)
 
   virtual bool AttributeAffectsRendering(int32_t aNameSpaceID,
                                          nsAtom* aAttribute) const override;
--- a/dom/svg/SVGIRect.h
+++ b/dom/svg/SVGIRect.h
@@ -15,17 +15,17 @@
 
 class nsIContent;
 
 namespace mozilla {
 namespace dom {
 
 class SVGIRect : public nsISupports, public nsWrapperCache {
  public:
-  virtual ~SVGIRect() {}
+  virtual ~SVGIRect() = default;
 
   JSObject* WrapObject(JSContext* aCx,
                        JS::Handle<JSObject*> aGivenProto) override {
     return SVGRect_Binding::Wrap(aCx, this, aGivenProto);
   }
 
   virtual nsIContent* GetParentObject() const = 0;
 
--- a/dom/svg/SVGIntegerPairSMILType.h
+++ b/dom/svg/SVGIntegerPairSMILType.h
@@ -36,14 +36,14 @@ class SVGIntegerPairSMILType : public SM
   virtual nsresult ComputeDistance(const SMILValue& aFrom, const SMILValue& aTo,
                                    double& aDistance) const override;
   virtual nsresult Interpolate(const SMILValue& aStartVal,
                                const SMILValue& aEndVal, double aUnitDistance,
                                SMILValue& aResult) const override;
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGIntegerPairSMILType() {}
+  constexpr SVGIntegerPairSMILType() = default;
 };
 
 }  // namespace mozilla
 
 #endif  // MOZILLA_SVGINTEGERPAIRSMILTYPE_H_
--- a/dom/svg/SVGLengthList.h
+++ b/dom/svg/SVGLengthList.h
@@ -34,18 +34,18 @@ class DOMSVGLengthList;
  * The DOM wrapper class for this class is DOMSVGLengthList.
  */
 class SVGLengthList {
   friend class dom::DOMSVGLength;
   friend class dom::DOMSVGLengthList;
   friend class SVGAnimatedLengthList;
 
  public:
-  SVGLengthList() {}
-  ~SVGLengthList() {}
+  SVGLengthList() = default;
+  ~SVGLengthList() = default;
 
   // Only methods that don't make/permit modification to this list are public.
   // Only our friend classes can access methods that may change us.
 
   /// This may return an incomplete string on OOM, but that's acceptable.
   void GetValueAsString(nsAString& aValue) const;
 
   bool IsEmpty() const { return mLengths.IsEmpty(); }
--- a/dom/svg/SVGLengthListSMILType.h
+++ b/dom/svg/SVGLengthListSMILType.h
@@ -86,14 +86,14 @@ class SVGLengthListSMILType : public SMI
   virtual nsresult ComputeDistance(const SMILValue& aFrom, const SMILValue& aTo,
                                    double& aDistance) const override;
   virtual nsresult Interpolate(const SMILValue& aStartVal,
                                const SMILValue& aEndVal, double aUnitDistance,
                                SMILValue& aResult) const override;
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGLengthListSMILType() {}
+  constexpr SVGLengthListSMILType() = default;
 };
 
 }  // namespace mozilla
 
 #endif  // MOZILLA_SVGLENGTHLISTSMILTYPE_H_
--- a/dom/svg/SVGMatrix.h
+++ b/dom/svg/SVGMatrix.h
@@ -58,17 +58,17 @@ class SVGMatrix final : public nsWrapper
    * Ctor for SVGMatrix objects that belong to a DOMSVGTransform.
    */
   explicit SVGMatrix(DOMSVGTransform& aTransform) : mTransform(&aTransform) {}
 
   /**
    * Ctors for SVGMatrix objects created independently of a DOMSVGTransform.
    */
   // Default ctor for gfxMatrix will produce identity mx
-  SVGMatrix() {}
+  SVGMatrix() = default;
 
   explicit SVGMatrix(const gfxMatrix& aMatrix) : mMatrix(aMatrix) {}
 
   const gfxMatrix& GetMatrix() const {
     return mTransform ? mTransform->Matrixgfx() : mMatrix;
   }
 
   // WebIDL
@@ -98,17 +98,17 @@ class SVGMatrix final : public nsWrapper
   already_AddRefed<SVGMatrix> RotateFromVector(float x, float y,
                                                ErrorResult& aRv);
   already_AddRefed<SVGMatrix> FlipX();
   already_AddRefed<SVGMatrix> FlipY();
   already_AddRefed<SVGMatrix> SkewX(float angle, ErrorResult& rv);
   already_AddRefed<SVGMatrix> SkewY(float angle, ErrorResult& rv);
 
  private:
-  ~SVGMatrix() {}
+  ~SVGMatrix() = default;
 
   void SetMatrix(const gfxMatrix& aMatrix) {
     if (mTransform) {
       mTransform->SetMatrix(aMatrix);
     } else {
       mMatrix = aMatrix;
     }
   }
--- a/dom/svg/SVGMotionSMILType.h
+++ b/dom/svg/SVGMotionSMILType.h
@@ -66,14 +66,14 @@ class SVGMotionSMILType : public SMILTyp
   // Used to generate a SMILValue for the point at the given distance along
   // the given path.
   static SMILValue ConstructSMILValue(Path* aPath, float aDist,
                                       RotateType aRotateType,
                                       float aRotateAngle);
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGMotionSMILType() {}
+  constexpr SVGMotionSMILType() = default;
 };
 
 }  // namespace mozilla
 
 #endif  // MOZILLA_SVGMOTIONSMILTYPE_H_
--- a/dom/svg/SVGNumberList.h
+++ b/dom/svg/SVGNumberList.h
@@ -32,18 +32,18 @@ class DOMSVGNumberList;
  * The DOM wrapper class for this class is DOMSVGNumberList.
  */
 class SVGNumberList {
   friend class dom::DOMSVGNumber;
   friend class dom::DOMSVGNumberList;
   friend class SVGAnimatedNumberList;
 
  public:
-  SVGNumberList() {}
-  ~SVGNumberList() {}
+  SVGNumberList() = default;
+  ~SVGNumberList() = default;
 
   // Only methods that don't make/permit modification to this list are public.
   // Only our friend classes can access methods that may change us.
 
   /// This may return an incomplete string on OOM, but that's acceptable.
   void GetValueAsString(nsAString& aValue) const;
 
   bool IsEmpty() const { return mNumbers.IsEmpty(); }
--- a/dom/svg/SVGNumberListSMILType.h
+++ b/dom/svg/SVGNumberListSMILType.h
@@ -40,14 +40,14 @@ class SVGNumberListSMILType : public SMI
   virtual nsresult ComputeDistance(const SMILValue& aFrom, const SMILValue& aTo,
                                    double& aDistance) const override;
   virtual nsresult Interpolate(const SMILValue& aStartVal,
                                const SMILValue& aEndVal, double aUnitDistance,
                                SMILValue& aResult) const override;
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGNumberListSMILType() {}
+  constexpr SVGNumberListSMILType() = default;
 };
 
 }  // namespace mozilla
 
 #endif  // MOZILLA_SVGNUMBERLISTSMILTYPE_H_
--- a/dom/svg/SVGNumberPairSMILType.h
+++ b/dom/svg/SVGNumberPairSMILType.h
@@ -33,14 +33,14 @@ class SVGNumberPairSMILType : public SMI
   virtual nsresult ComputeDistance(const SMILValue& aFrom, const SMILValue& aTo,
                                    double& aDistance) const override;
   virtual nsresult Interpolate(const SMILValue& aStartVal,
                                const SMILValue& aEndVal, double aUnitDistance,
                                SMILValue& aResult) const override;
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGNumberPairSMILType() {}
+  constexpr SVGNumberPairSMILType() = default;
 };
 
 }  // namespace mozilla
 
 #endif  // MOZILLA_SVGNUMBERPAIRSMILTYPE_H_
--- a/dom/svg/SVGOrientSMILType.h
+++ b/dom/svg/SVGOrientSMILType.h
@@ -53,14 +53,14 @@ class SVGOrientSMILType : public SMILTyp
   virtual nsresult ComputeDistance(const SMILValue& aFrom, const SMILValue& aTo,
                                    double& aDistance) const override;
   virtual nsresult Interpolate(const SMILValue& aStartVal,
                                const SMILValue& aEndVal, double aUnitDistance,
                                SMILValue& aResult) const override;
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGOrientSMILType() {}
+  constexpr SVGOrientSMILType() = default;
 };
 
 }  // namespace mozilla
 
 #endif  // MOZILLA_SVGORIENTSMILTYPE_H_
--- a/dom/svg/SVGPathData.h
+++ b/dom/svg/SVGPathData.h
@@ -86,18 +86,18 @@ class SVGPathData {
   typedef gfx::PathBuilder PathBuilder;
   typedef gfx::FillRule FillRule;
   typedef gfx::Float Float;
   typedef gfx::CapStyle CapStyle;
 
  public:
   typedef const float* const_iterator;
 
-  SVGPathData() {}
-  ~SVGPathData() {}
+  SVGPathData() = default;
+  ~SVGPathData() = default;
 
   // Only methods that don't make/permit modification to this list are public.
   // Only our friend classes can access methods that may change us.
 
   /// This may return an incomplete string on OOM, but that's acceptable.
   void GetValueAsString(nsAString& aValue) const;
 
   bool IsEmpty() const { return mData.IsEmpty(); }
--- a/dom/svg/SVGPathSegListSMILType.h
+++ b/dom/svg/SVGPathSegListSMILType.h
@@ -43,14 +43,14 @@ class SVGPathSegListSMILType : public SM
   virtual nsresult ComputeDistance(const SMILValue& aFrom, const SMILValue& aTo,
                                    double& aDistance) const override;
   virtual nsresult Interpolate(const SMILValue& aStartVal,
                                const SMILValue& aEndVal, double aUnitDistance,
                                SMILValue& aResult) const override;
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGPathSegListSMILType() {}
+  constexpr SVGPathSegListSMILType() = default;
 };
 
 }  // namespace mozilla
 
 #endif  // MOZILLA_SVGPATHSEGLISTSMILTYPE_H_
--- a/dom/svg/SVGPathSegUtils.h
+++ b/dom/svg/SVGPathSegUtils.h
@@ -68,17 +68,17 @@ struct SVGPathTraversalState {
  *
  * The DOM wrapper classes for encoded path segments (data contained in
  * instances of SVGPathData) is DOMSVGPathSeg and its sub-classes. Note that
  * there are multiple different DOM classes for path segs - one for each of the
  * 19 SVG 1.1 segment types.
  */
 class SVGPathSegUtils {
  private:
-  SVGPathSegUtils() {}  // private to prevent instances
+  SVGPathSegUtils() = default;  // private to prevent instances
 
  public:
   static void GetValueAsString(const float* aSeg, nsAString& aValue);
 
   /**
    * Encode a segment type enum to a float.
    *
    * At some point in the future we will likely want to encode other
--- a/dom/svg/SVGPointList.h
+++ b/dom/svg/SVGPointList.h
@@ -32,18 +32,18 @@ class nsISVGPoint;
  */
 class SVGPointList {
   friend class mozilla::nsISVGPoint;
   friend class SVGAnimatedPointList;
   friend class DOMSVGPointList;
   friend class DOMSVGPoint;
 
  public:
-  SVGPointList() {}
-  ~SVGPointList() {}
+  SVGPointList() = default;
+  ~SVGPointList() = default;
 
   // Only methods that don't make/permit modification to this list are public.
   // Only our friend classes can access methods that may change us.
 
   /// This may return an incomplete string on OOM, but that's acceptable.
   void GetValueAsString(nsAString& aValue) const;
 
   bool IsEmpty() const { return mItems.IsEmpty(); }
--- a/dom/svg/SVGPointListSMILType.h
+++ b/dom/svg/SVGPointListSMILType.h
@@ -40,14 +40,14 @@ class SVGPointListSMILType : public SMIL
   virtual nsresult ComputeDistance(const SMILValue& aFrom, const SMILValue& aTo,
                                    double& aDistance) const override;
   virtual nsresult Interpolate(const SMILValue& aStartVal,
                                const SMILValue& aEndVal, double aUnitDistance,
                                SMILValue& aResult) const override;
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGPointListSMILType() {}
+  constexpr SVGPointListSMILType() = default;
 };
 
 }  // namespace mozilla
 
 #endif  // MOZILLA_SVGPOINTLISTSMILTYPE_H_
--- a/dom/svg/SVGRect.h
+++ b/dom/svg/SVGRect.h
@@ -41,17 +41,17 @@ class SVGRect final : public SVGIRect {
 
   float Height() const final { return mHeight; }
 
   void SetHeight(float aHeight, ErrorResult& aRv) final { mHeight = aHeight; }
 
   virtual nsIContent* GetParentObject() const override { return mParent; }
 
  protected:
-  ~SVGRect() {}
+  ~SVGRect() = default;
 
   nsCOMPtr<nsIContent> mParent;
   float mX, mY, mWidth, mHeight;
 };
 
 }  // namespace dom
 }  // namespace mozilla
 
--- a/dom/svg/SVGSVGElement.h
+++ b/dom/svg/SVGSVGElement.h
@@ -60,17 +60,17 @@ class DOMSVGTranslatePoint final : publi
   virtual already_AddRefed<nsISVGPoint> MatrixTransform(
       SVGMatrix& matrix) override;
 
   virtual nsISupports* GetParentObject() override;
 
   RefPtr<SVGSVGElement> mElement;
 
  private:
-  ~DOMSVGTranslatePoint() {}
+  ~DOMSVGTranslatePoint() = default;
 };
 
 typedef SVGViewportElement SVGSVGElementBase;
 
 class SVGSVGElement final : public SVGSVGElementBase {
   friend class ::nsSVGOuterSVGFrame;
   friend class mozilla::SVGFragmentIdentifier;
   friend class mozilla::AutoSVGViewHandler;
--- a/dom/svg/SVGStringList.h
+++ b/dom/svg/SVGStringList.h
@@ -17,17 +17,17 @@ namespace mozilla {
  *
  * The DOM wrapper class for this class is DOMSVGStringList.
  */
 class SVGStringList {
   friend class DOMSVGStringList;
 
  public:
   SVGStringList() : mIsSet(false), mIsCommaSeparated(false) {}
-  ~SVGStringList() {}
+  ~SVGStringList() = default;
 
   void SetIsCommaSeparated(bool aIsCommaSeparated) {
     mIsCommaSeparated = aIsCommaSeparated;
   }
   nsresult SetValue(const nsAString& aValue);
 
   void Clear() {
     mStrings.Clear();
--- a/dom/svg/SVGTests.h
+++ b/dom/svg/SVGTests.h
@@ -100,17 +100,17 @@ class SVGTests : public nsISupports {
 
   virtual SVGElement* AsSVGElement() = 0;
 
   const SVGElement* AsSVGElement() const {
     return const_cast<SVGTests*>(this)->AsSVGElement();
   }
 
  protected:
-  virtual ~SVGTests() {}
+  virtual ~SVGTests() = default;
 
  private:
   enum { FEATURES, EXTENSIONS, LANGUAGE };
   SVGStringList mStringListAttributes[3];
   static nsStaticAtom* const sStringListNames[3];
 };
 
 NS_DEFINE_STATIC_IID_ACCESSOR(SVGTests, MOZILLA_DOMSVGTESTS_IID)
--- a/dom/svg/SVGTransformList.h
+++ b/dom/svg/SVGTransformList.h
@@ -27,18 +27,18 @@ class DOMSVGTransform;
  * The DOM wrapper class for this class is DOMSVGTransformList.
  */
 class SVGTransformList {
   friend class SVGAnimatedTransformList;
   friend class DOMSVGTransformList;
   friend class dom::DOMSVGTransform;
 
  public:
-  SVGTransformList() {}
-  ~SVGTransformList() {}
+  SVGTransformList() = default;
+  ~SVGTransformList() = default;
 
   // Only methods that don't make/permit modification to this list are public.
   // Only our friend classes can access methods that may change us.
 
   /// This may return an incomplete string on OOM, but that's acceptable.
   void GetValueAsString(nsAString& aValue) const;
 
   bool IsEmpty() const { return mItems.IsEmpty(); }
--- a/dom/svg/SVGTransformListSMILType.h
+++ b/dom/svg/SVGTransformListSMILType.h
@@ -111,14 +111,14 @@ class SVGTransformListSMILType : public 
                                   SMILValue& aValue);
   static bool AppendTransforms(const SVGTransformList& aList,
                                SMILValue& aValue);
   static bool GetTransforms(const SMILValue& aValue,
                             FallibleTArray<SVGTransform>& aTransforms);
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGTransformListSMILType() {}
+  constexpr SVGTransformListSMILType() = default;
 };
 
 }  // end namespace mozilla
 
 #endif  // SVGLISTTRANSFORMSMILTYPE_H_
--- a/dom/svg/SVGTransformableElement.h
+++ b/dom/svg/SVGTransformableElement.h
@@ -22,17 +22,17 @@ class SVGGraphicsElement;
 class SVGMatrix;
 class SVGIRect;
 struct SVGBoundingBoxOptions;
 
 class SVGTransformableElement : public SVGElement {
  public:
   explicit SVGTransformableElement(already_AddRefed<dom::NodeInfo>&& aNodeInfo)
       : SVGElement(std::move(aNodeInfo)) {}
-  virtual ~SVGTransformableElement() {}
+  virtual ~SVGTransformableElement() = default;
 
   virtual nsresult Clone(dom::NodeInfo*, nsINode** aResult) const override = 0;
 
   // WebIDL
   already_AddRefed<DOMSVGAnimatedTransformList> Transform();
   SVGElement* GetNearestViewportElement();
   SVGElement* GetFarthestViewportElement();
   MOZ_CAN_RUN_SCRIPT
--- a/dom/svg/SVGViewBoxSMILType.h
+++ b/dom/svg/SVGViewBoxSMILType.h
@@ -33,14 +33,14 @@ class SVGViewBoxSMILType : public SMILTy
   virtual nsresult ComputeDistance(const SMILValue& aFrom, const SMILValue& aTo,
                                    double& aDistance) const override;
   virtual nsresult Interpolate(const SMILValue& aStartVal,
                                const SMILValue& aEndVal, double aUnitDistance,
                                SMILValue& aResult) const override;
 
  private:
   // Private constructor: prevent instances beyond my singleton.
-  constexpr SVGViewBoxSMILType() {}
+  constexpr SVGViewBoxSMILType() = default;
 };
 
 }  // namespace mozilla
 
 #endif  // MOZILLA_SVGVIEWBOXSMILTYPE_H_