fixed call to ToInteger() to respect the error code

content/html/content/src/nsHTMLFontElement.cpp

@@ -142,12 +142,16 @@ nsHTMLFontElement::StringToAttribute(nsIAtom* aAttribute,
       (aAttribute == nsHTMLAtoms::pointSize) ||
       (aAttribute == nsHTMLAtoms::fontWeight)) {
     nsAutoString tmp(aValue);
+      //rickg: fixed flaw where ToInteger error code was not being checked.
+      //       This caused wrong default value for font size.
     PRInt32 ec, v = tmp.ToInteger(&ec);
-    tmp.CompressWhitespace(PR_TRUE, PR_FALSE);
-    PRUnichar ch = tmp.First();
-    aResult.SetIntValue(v, ((ch == '+') || (ch == '-')) ?
-                        eHTMLUnit_Integer : eHTMLUnit_Enumerated);
-    return NS_CONTENT_ATTR_HAS_VALUE;
+    if(NS_SUCCEEDED(ec)){
+      tmp.CompressWhitespace(PR_TRUE, PR_FALSE);
+      PRUnichar ch = tmp.First();
+      aResult.SetIntValue(v, ((ch == '+') || (ch == '-')) ?
+                          eHTMLUnit_Integer : eHTMLUnit_Enumerated);
+      return NS_CONTENT_ATTR_HAS_VALUE;
+    }
   }
   if (aAttribute == nsHTMLAtoms::color) {
     nsGenericHTMLElement::ParseColor(aValue, aResult);

layout/html/content/src/nsHTMLFontElement.cpp

@@ -142,12 +142,16 @@ nsHTMLFontElement::StringToAttribute(nsIAtom* aAttribute,
       (aAttribute == nsHTMLAtoms::pointSize) ||
       (aAttribute == nsHTMLAtoms::fontWeight)) {
     nsAutoString tmp(aValue);
+      //rickg: fixed flaw where ToInteger error code was not being checked.
+      //       This caused wrong default value for font size.
     PRInt32 ec, v = tmp.ToInteger(&ec);
-    tmp.CompressWhitespace(PR_TRUE, PR_FALSE);
-    PRUnichar ch = tmp.First();
-    aResult.SetIntValue(v, ((ch == '+') || (ch == '-')) ?
-                        eHTMLUnit_Integer : eHTMLUnit_Enumerated);
-    return NS_CONTENT_ATTR_HAS_VALUE;
+    if(NS_SUCCEEDED(ec)){
+      tmp.CompressWhitespace(PR_TRUE, PR_FALSE);
+      PRUnichar ch = tmp.First();
+      aResult.SetIntValue(v, ((ch == '+') || (ch == '-')) ?
+                          eHTMLUnit_Integer : eHTMLUnit_Enumerated);
+      return NS_CONTENT_ATTR_HAS_VALUE;
+    }
   }
   if (aAttribute == nsHTMLAtoms::color) {
     nsGenericHTMLElement::ParseColor(aValue, aResult);