Backed out changeset fc1c18cda24b (bug 1597891) for causing oranges

Oranges: https://treeherder.mozilla.org/#/jobs?repo=comm-central&selectedJob=279022667&revision=fc1c18cda24b0a157d4b6d71c401bf4b5146b17e

mailnews/mime/src/mimemoz2.cpp

@@ -1772,32 +1772,43 @@ extern "C" char *MimeGetStringByName(const char16_t *stringName) {
 void ResetChannelCharset(MimeObject *obj) {
   if (obj->options && obj->options->stream_closure &&
       obj->options->default_charset && obj->headers) {
-    // The previous version of this code would have entered an infinite
-    // loop. But it never showed up, so it's not clear that it is ever called.
-    // See bug #1597891.
-    MOZ_DIAGNOSTIC_ASSERT(false, "Should never get here (see bug 1597891)");
-
     mime_stream_data *msd = (mime_stream_data *)(obj->options->stream_closure);
     char *ct = MimeHeaders_get(obj->headers, HEADER_CONTENT_TYPE, false, false);
-    if (ct && msd && msd->channel) {
-      char *cSet = MimeHeaders_get_parameter(ct, "charset", nullptr, nullptr);
-      if (cSet) {
-        // First, setup the channel.
+    if ((ct) && (msd) && (msd->channel)) {
+      char *ptr = strstr(ct, "charset=");
+      if (ptr) {
+        // First, setup the channel!
         msd->channel->SetContentType(nsDependentCString(ct));
 
         // Second, if this is a Save As operation, then we need to convert
-        // to override the output charset.
-        if (msd->format_out == nsMimeOutput::nsMimeMessageSaveAs) {
+        // to override the output charset!
+        mime_stream_data *msd = GetMSD(obj->options);
+        if ((msd) && (msd->format_out == nsMimeOutput::nsMimeMessageSaveAs)) {
+          // Extract the charset alone
+          char *cSet = nullptr;
+          if (*(ptr + 8) == '"')
+            cSet = strdup(ptr + 9);
+          else
+            cSet = strdup(ptr + 8);
+          if (cSet) {
+            char *ptr2 = cSet;
+            while ((*cSet) && (*cSet != ' ') && (*cSet != ';') &&
+                   (*cSet != '\r') && (*cSet != '\n') && (*cSet != '"'))
+              ptr2++;
+
+            if (*cSet) {
               PR_FREEIF(obj->options->default_charset);
-          obj->options->default_charset = cSet;
-          cSet = nullptr;  // Ownership was transferred.
+              obj->options->default_charset = strdup(cSet);
               obj->options->override_charset = true;
             }
+
             PR_FREEIF(cSet);
           }
         }
+      }
       PR_FREEIF(ct);
     }
+  }
 }
 
 ////////////////////////////////////////////////////////////