Bug 1392529 - Follow-up (take 2): Simplification. r=aceman DONTBUILD
authorJorg K <jorgk@jorgk.com>
Tue, 22 Aug 2017 21:33:13 +0200
changeset 22022 be1c2f3714964d16a85cb2d238831fa789af539a
parent 22021 6136a4dd0f4d6a4e5797b8868c2f140389ec100b
child 22023 89f6848383e34cf869621b12001ddffa60ec8bc2
push id13442
push usermozilla@jorgk.com
push dateTue, 22 Aug 2017 19:39:13 +0000
treeherdercomm-central@be1c2f371496 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersaceman
bugs1392529
Bug 1392529 - Follow-up (take 2): Simplification. r=aceman DONTBUILD
mailnews/mime/emitters/nsMimeBaseEmitter.cpp
--- a/mailnews/mime/emitters/nsMimeBaseEmitter.cpp
+++ b/mailnews/mime/emitters/nsMimeBaseEmitter.cpp
@@ -868,19 +868,17 @@ nsMimeBaseEmitter::WriteHeaderFieldHTMLP
       ( (mFormat == nsMimeOutput::nsMimeMessagePrintOutput) && (mFirstHeaders) )
      )
      /* DO NOTHING */ ;   // rhp: Do nothing...leaving the conditional like this so its
                           //      easier to see the logic of what is going on.
   else {
     mHTMLHeaders.Append("<br><fieldset class=\"mimeAttachmentHeader\">");
     if (!name.IsEmpty()) {
       mHTMLHeaders.Append("<legend class=\"mimeAttachmentHeaderName\">");
-      nsCString escapedName;
-      nsAppendEscapedHTML(name, escapedName);
-      mHTMLHeaders.Append(escapedName);
+      nsAppendEscapedHTML(name, mHTMLHeaders);
       mHTMLHeaders.Append("</legend>");
     }
     mHTMLHeaders.Append("</fieldset>");
   }
 
   mFirstHeaders = false;
   return NS_OK;
 }