Bug 1299102 - swap NS_PRECONDITION with MOZ_ASSERT in TransformString. r?jfkthame draft
authorAndi-Bogdan Postelnicu <bpostelnicu@mozilla.com>
Wed, 31 Aug 2016 12:15:31 +0300
changeset 407916 470922744b0e6642a1b1cc7dff0d7b4ca4683dca
parent 407237 26e22af660e543ebb69930f082188b69ec756185
child 529991 7e96c8f6fe550bf8f2b15f8dd1a5aa2aea39e28a
push id28089
push userbmo:bpostelnicu@mozilla.com
push dateWed, 31 Aug 2016 09:16:02 +0000
reviewersjfkthame
bugs1299102
milestone51.0a1
Bug 1299102 - swap NS_PRECONDITION with MOZ_ASSERT in TransformString. r?jfkthame MozReview-Commit-ID: 7X6QA8qazKL
layout/generic/nsTextRunTransformations.cpp
--- a/layout/generic/nsTextRunTransformations.cpp
+++ b/layout/generic/nsTextRunTransformations.cpp
@@ -281,18 +281,18 @@ nsCaseTransformTextRunFactory::Transform
     nsTArray<bool>& aCharsToMergeArray,
     nsTArray<bool>& aDeletedCharsArray,
     const nsTransformedTextRun* aTextRun,
     uint32_t aOffsetInTextRun,
     nsTArray<uint8_t>* aCanBreakBeforeArray,
     nsTArray<RefPtr<nsTransformedCharStyle>>* aStyleArray)
 {
   bool auxiliaryOutputArrays = aCanBreakBeforeArray && aStyleArray;
-  NS_PRECONDITION(!auxiliaryOutputArrays || aTextRun,
-                  "text run must be provided to use aux output arrays");
+  MOZ_ASSERT(!auxiliaryOutputArrays || aTextRun,
+      "text run must be provided to use aux output arrays");
 
   uint32_t length = aString.Length();
   const char16_t* str = aString.BeginReading();
 
   bool mergeNeeded = false;
 
   bool capitalizeDutchIJ = false;
   bool prevIsLetter = false;