Bug 1729329 - Mark txPushNewContext::addSort as infallible. r=farre

Differential Revision: https://phabricator.services.mozilla.com/D124660

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) {

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;

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);
 }