#107905

r=mkaply, sr=attinasi
OS/2 only - don't shortcut font computations for fixed with fonts

layout/generic/nsTextFrame.cpp

@@ -4395,12 +4395,16 @@ nsTextFrame::MeasureText(nsIPresContext*          aPresContext,
   PRBool  endsInNewline = PR_FALSE;
   PRBool  justDidFirstLetter = PR_FALSE;
   nsTextDimensions dimensions, lastWordDimensions;
-#if defined(_WIN32) || defined(XP_OS2)
+#if defined(_WIN32)
   PRBool  measureTextRuns = !aTextData.mComputeMaxWordWidth && !aTs.mPreformatted &&
                             !aTs.mSmallCaps && !aTs.mWordSpacing && !aTs.mLetterSpacing &&
                             aTextData.mWrapping;
   // Don't measure text runs with letter spacing active, it doesn't work
   // it also doesn't work if we are not word-wrapping (bug 42832)
+#elif defined(XP_OS2)
+  PRBool  measureTextRuns = !aTextData.mComputeMaxWordWidth && !aTs.mSmallCaps &&
+                            !aTs.mWordSpacing && !aTs.mLetterSpacing &&
+                            aTextData.mWrapping;
 #else
   PRBool  measureTextRuns = PR_FALSE;
 #endif

layout/html/base/src/nsTextFrame.cpp

@@ -4395,12 +4395,16 @@ nsTextFrame::MeasureText(nsIPresContext*          aPresContext,
   PRBool  endsInNewline = PR_FALSE;
   PRBool  justDidFirstLetter = PR_FALSE;
   nsTextDimensions dimensions, lastWordDimensions;
-#if defined(_WIN32) || defined(XP_OS2)
+#if defined(_WIN32)
   PRBool  measureTextRuns = !aTextData.mComputeMaxWordWidth && !aTs.mPreformatted &&
                             !aTs.mSmallCaps && !aTs.mWordSpacing && !aTs.mLetterSpacing &&
                             aTextData.mWrapping;
   // Don't measure text runs with letter spacing active, it doesn't work
   // it also doesn't work if we are not word-wrapping (bug 42832)
+#elif defined(XP_OS2)
+  PRBool  measureTextRuns = !aTextData.mComputeMaxWordWidth && !aTs.mSmallCaps &&
+                            !aTs.mWordSpacing && !aTs.mLetterSpacing &&
+                            aTextData.mWrapping;
 #else
   PRBool  measureTextRuns = PR_FALSE;
 #endif