Fixed document.writeln problem from http://www.legomindspring.com

content/html/document/src/nsHTMLDocument.cpp

@@ -737,22 +737,21 @@ nsHTMLDocument::Write(JSContext *cx, jsval *argv, PRUint32 argc)
   
   if (argc > 0) {
     PRUint32 index;
+    nsAutoString str;
     for (index = 0; index < argc; index++) {
-      nsAutoString str;
       JSString *jsstring = JS_ValueToString(cx, argv[index]);
       
       if (nsnull != jsstring) {
-        str.SetString(JS_GetStringChars(jsstring));
+        str.Append(JS_GetStringChars(jsstring));
       }
       else {
-        str.SetString("");   // Should this really be null?? 
-      }
-      
-      result = mParser->Parse(str, PR_TRUE);
-      if (NS_OK != result) {
-        return result;
+        str.Append("");   // Should this really be null?? 
       }
     }
+    result = mParser->Parse(str, PR_TRUE);
+    if (NS_OK != result) {
+      return result;
+    }
   }
   
   return result;

layout/html/document/src/nsHTMLDocument.cpp

@@ -737,22 +737,21 @@ nsHTMLDocument::Write(JSContext *cx, jsval *argv, PRUint32 argc)
   
   if (argc > 0) {
     PRUint32 index;
+    nsAutoString str;
     for (index = 0; index < argc; index++) {
-      nsAutoString str;
       JSString *jsstring = JS_ValueToString(cx, argv[index]);
       
       if (nsnull != jsstring) {
-        str.SetString(JS_GetStringChars(jsstring));
+        str.Append(JS_GetStringChars(jsstring));
       }
       else {
-        str.SetString("");   // Should this really be null?? 
-      }
-      
-      result = mParser->Parse(str, PR_TRUE);
-      if (NS_OK != result) {
-        return result;
+        str.Append("");   // Should this really be null?? 
       }
     }
+    result = mParser->Parse(str, PR_TRUE);
+    if (NS_OK != result) {
+      return result;
+    }
   }
   
   return result;