diff --git a/dom/xslt/xslt/txInstructions.cpp b/dom/xslt/xslt/txInstructions.cpp
index 87bd181ca785..52bd24b8059e 100644
--- a/dom/xslt/xslt/txInstructions.cpp
+++ b/dom/xslt/xslt/txInstructions.cpp
@@ -559,7 +559,7 @@ nsresult txPushNewContext::execute(txExecutionState& aEs) {
   return NS_OK;
 }
 
-nsresult txPushNewContext::addSort(UniquePtr<Expr>&& aSelectExpr,
+void txPushNewContext::addSort(UniquePtr<Expr>&& aSelectExpr,
                                    UniquePtr<Expr>&& aLangExpr,
                                    UniquePtr<Expr>&& aDataTypeExpr,
                                    UniquePtr<Expr>&& aOrderExpr,
@@ -571,7 +571,6 @@ nsresult txPushNewContext::addSort(UniquePtr<Expr>&& aSelectExpr,
   key->mDataTypeExpr = std::move(aDataTypeExpr);
   key->mOrderExpr = std::move(aOrderExpr);
   key->mCaseOrderExpr = std::move(aCaseOrderExpr);
-  return NS_OK;
 }
 
 nsresult txPushNullTemplateRule::execute(txExecutionState& aEs) {
diff --git a/dom/xslt/xslt/txInstructions.h b/dom/xslt/xslt/txInstructions.h
index 97e655075f85..61e7ec956c79 100644
--- a/dom/xslt/xslt/txInstructions.h
+++ b/dom/xslt/xslt/txInstructions.h
@@ -229,11 +229,11 @@ class txPushNewContext : public txInstruction {
 
   TX_DECL_TXINSTRUCTION
 
-  nsresult addSort(mozilla::UniquePtr<Expr>&& aSelectExpr,
-                   mozilla::UniquePtr<Expr>&& aLangExpr,
-                   mozilla::UniquePtr<Expr>&& aDataTypeExpr,
-                   mozilla::UniquePtr<Expr>&& aOrderExpr,
-                   mozilla::UniquePtr<Expr>&& aCaseOrderExpr);
+  void addSort(mozilla::UniquePtr<Expr>&& aSelectExpr,
+               mozilla::UniquePtr<Expr>&& aLangExpr,
+               mozilla::UniquePtr<Expr>&& aDataTypeExpr,
+               mozilla::UniquePtr<Expr>&& aOrderExpr,
+               mozilla::UniquePtr<Expr>&& aCaseOrderExpr);
 
   struct SortKey {
     mozilla::UniquePtr<Expr> mSelectExpr;
diff --git a/dom/xslt/xslt/txStylesheetCompileHandlers.cpp b/dom/xslt/xslt/txStylesheetCompileHandlers.cpp
index 224437de4c89..b2f8e9d874fa 100644
--- a/dom/xslt/xslt/txStylesheetCompileHandlers.cpp
+++ b/dom/xslt/xslt/txStylesheetCompileHandlers.cpp
@@ -1882,10 +1882,9 @@ static nsresult txFnStartSort(int32_t aNamespaceID, nsAtom* aLocalName,
                   caseOrder);
   NS_ENSURE_SUCCESS(rv, rv);
 
-  rv = aState.mSorter->addSort(std::move(select), std::move(lang),
-                               std::move(dataType), std::move(order),
-                               std::move(caseOrder));
-  NS_ENSURE_SUCCESS(rv, rv);
+  aState.mSorter->addSort(std::move(select), std::move(lang),
+                          std::move(dataType), std::move(order),
+                          std::move(caseOrder));
 
   return aState.pushHandlerTable(gTxIgnoreHandler);
 }