reflow comments in platform/mhtml

third_party/WebKit/Source/platform/mhtml/MHTMLParser.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 24 matching lines @@
 #include "platform/text/QuotedPrintable.h"
 #include "wtf/HashMap.h"
 #include "wtf/text/Base64.h"
 #include "wtf/text/StringBuilder.h"
 #include "wtf/text/StringConcatenate.h"
 #include "wtf/text/StringHash.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
-// This class is a limited MIME parser used to parse the MIME headers of MHTML files.
+// This class is a limited MIME parser used to parse the MIME headers of MHTML
+// files.
 class MIMEHeader : public GarbageCollectedFinalized<MIMEHeader> {
  public:
   static MIMEHeader* create() { return new MIMEHeader; }
 
   enum Encoding {
     QuotedPrintable,
     Base64,
     EightBit,
     SevenBit,
     Binary,
@@ skipping 177 matching lines @@
   skipLinesUntilBoundaryFound(m_lineReader, header->endOfPartBoundary());
 
   bool endOfArchive = false;
   while (!endOfArchive) {
     MIMEHeader* resourceHeader = MIMEHeader::parseHeader(&m_lineReader);
     if (!resourceHeader) {
       DVLOG(1) << "Failed to parse MHTML, invalid MIME header.";
       return false;
     }
     if (resourceHeader->contentType() == "multipart/alternative") {
-      // Ignore IE nesting which makes little sense (IE seems to nest only some of the frames).
+      // Ignore IE nesting which makes little sense (IE seems to nest only some
+      // of the frames).
       if (!parseArchiveWithHeader(resourceHeader, resources)) {
         DVLOG(1) << "Failed to parse MHTML subframe.";
         return false;
       }
       skipLinesUntilBoundaryFound(m_lineReader, header->endOfPartBoundary());
       continue;
     }
 
     ArchiveResource* resource =
         parseNextPart(*resourceHeader, header->endOfPartBoundary(),
@@ skipping 52 matching lines @@
     }
   } else {
     String line;
     while (!(line = m_lineReader.nextChunkAsUTF8StringWithLatin1Fallback())
                 .isNull()) {
       endOfArchiveReached = (line == endOfDocumentBoundary);
       if (checkBoundary && (line == endOfPartBoundary || endOfArchiveReached)) {
         endOfPartReached = true;
         break;
       }
-      // Note that we use line.utf8() and not line.ascii() as ascii turns special characters (such as tab, line-feed...) into '?'.
+      // Note that we use line.utf8() and not line.ascii() as ascii turns
+      // special characters (such as tab, line-feed...) into '?'.
       content->append(line.utf8().data(), line.length());
       if (contentTransferEncoding == MIMEHeader::QuotedPrintable) {
-        // The line reader removes the \r\n, but we need them for the content in this case as the QuotedPrintable decoder expects CR-LF terminated lines.
+        // The line reader removes the \r\n, but we need them for the content in
+        // this case as the QuotedPrintable decoder expects CR-LF terminated
+        // lines.
         content->append("\r\n", 2u);
       }
     }
   }
   if (!endOfPartReached && checkBoundary) {
     DVLOG(1) << "No boundary found for MHTML part.";
     return nullptr;
   }
 
   Vector<char> data;
@@ skipping 10 matching lines @@
     case MIMEHeader::EightBit:
     case MIMEHeader::SevenBit:
     case MIMEHeader::Binary:
       data.append(content->data(), content->size());
       break;
     default:
       DVLOG(1) << "Invalid encoding for MHTML part.";
       return nullptr;
   }
   RefPtr<SharedBuffer> contentBuffer = SharedBuffer::adoptVector(data);
-  // FIXME: the URL in the MIME header could be relative, we should resolve it if it is.
-  // The specs mentions 5 ways to resolve a URL: http://tools.ietf.org/html/rfc2557#section-5
+  // FIXME: the URL in the MIME header could be relative, we should resolve it
+  // if it is.  The specs mentions 5 ways to resolve a URL:
+  // http://tools.ietf.org/html/rfc2557#section-5
   // IE and Firefox (UNMht) seem to generate only absolute URLs.
   KURL location = KURL(KURL(), mimeHeader.contentLocation());
   return ArchiveResource::create(contentBuffer, location,
                                  mimeHeader.contentID(),
                                  AtomicString(mimeHeader.contentType()),
                                  AtomicString(mimeHeader.charset()));
 }
 
 // static
 KURL MHTMLParser::convertContentIDToURI(const String& contentID) {
   // This function is based primarily on an example from rfc2557 in section
   // 9.5, but also based on more normative parts of specs like:
-  // - rfc2557 - MHTML - section 8.3 - "Use of the Content-ID header and CID URLs"
+  // - rfc2557 - MHTML - section 8.3 - "Use of the Content-ID header and CID
+  //                                    URLs"
   // - rfc1738 - URL - section 4 (reserved scheme names;  includes "cid")
   // - rfc2387 - multipart/related - section 3.4 - "Syntax" (cid := msg-id)
   // - rfc0822 - msg-id = "<" addr-spec ">"; addr-spec = local-part "@" domain
 
   if (contentID.length() <= 2)
     return KURL();
 
   if (!contentID.startsWith('<') || !contentID.endsWith('>'))
     return KURL();
 
   StringBuilder uriBuilder;
   uriBuilder.append("cid:");
   uriBuilder.append(contentID, 1, contentID.length() - 2);
   return KURL(KURL(), uriBuilder.toString());
 }
 
 }  // namespace blink