--- a/editor/txmgr/tests/TestTXMgr.cpp
+++ b/editor/txmgr/tests/TestTXMgr.cpp
@@ -250,17 +250,17 @@ int32_t sAggregateBatchTestRedoOrderArr[
         127, 128, 129, 130, 131, 132, 133, 134, 135, 136, 137, 138, 139, 140,
         281, 282, 283, 284, 285, 286, 287, 288, 289, 290, 291, 292, 293, 294,
         295, 296, 297, 298, 299, 300, 301, 448, 449, 450, 451, 452, 453, 454,
         455, 456, 457, 458 };
 
 class TestTransaction : public nsITransaction
 {
 protected:
-  virtual ~TestTransaction() {}
+  virtual ~TestTransaction() = default;
 
 public:
 
   TestTransaction() {}
 
   NS_DECL_ISUPPORTS
 };
 
@@ -283,19 +283,17 @@ protected:
   int32_t mFlags;
 
 public:
 
   explicit SimpleTransaction(int32_t aFlags=NONE_FLAG)
     : mVal(++sConstructorCount), mFlags(aFlags)
   {}
 
-  virtual ~SimpleTransaction()
-  {
-  }
+  virtual ~SimpleTransaction() = default;
 
   NS_IMETHOD DoTransaction()
   {
     //
     // Make sure DoTransaction() is called in the order we expect!
     // Notice that we don't check to see if we go past the end of the array.
     // This is done on purpose since we want to crash if the order array is out
     // of date.
@@ -398,19 +396,17 @@ public:
     mNumber             = 1;
     mFlags              = aFlags & (~ALL_ERROR_FLAGS);
     mErrorFlags         = aFlags & ALL_ERROR_FLAGS;
     mTXMgr              = aTXMgr;
     mMaxLevel           = aMaxLevel;
     mNumChildrenPerNode = aNumChildrenPerNode;
   }
 
-  virtual ~AggregateTransaction()
-  {
-  }
+  virtual ~AggregateTransaction() = default;
 
   NS_IMETHOD DoTransaction()
   {
     if (mLevel >= mMaxLevel) {
       // Only leaf nodes can throw errors!
       mFlags |= mErrorFlags;
     }