Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1080)

Unified Diff: third_party/WebKit/Source/platform/mhtml/MHTMLArchive.cpp

Issue 2384063002: reflow comments in platform/mhtml (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/mhtml/MHTMLArchive.cpp
diff --git a/third_party/WebKit/Source/platform/mhtml/MHTMLArchive.cpp b/third_party/WebKit/Source/platform/mhtml/MHTMLArchive.cpp
index 449fa31926d947bad585ef210e2d7d1568225f3b..09d61918d13cea2522f5b5893fedf9f19981290f 100644
--- a/third_party/WebKit/Source/platform/mhtml/MHTMLArchive.cpp
+++ b/third_party/WebKit/Source/platform/mhtml/MHTMLArchive.cpp
@@ -66,8 +66,9 @@ MHTMLArchive::MHTMLArchive() {}
MHTMLArchive* MHTMLArchive::create(const KURL& url,
PassRefPtr<const SharedBuffer> data) {
- // MHTML pages can only be loaded from local URLs, http/https URLs, and content URLs(Android specific).
- // The latter is now allowed due to full sandboxing enforcement on MHTML pages.
+ // MHTML pages can only be loaded from local URLs, http/https URLs, and
+ // content URLs(Android specific). The latter is now allowed due to full
+ // sandboxing enforcement on MHTML pages.
if (!canLoadArchive(url))
return nullptr;
@@ -92,8 +93,9 @@ MHTMLArchive* MHTMLArchive::create(const KURL& url,
}
bool MHTMLArchive::canLoadArchive(const KURL& url) {
- // MHTML pages can only be loaded from local URLs, http/https URLs, and content URLs(Android specific).
- // The latter is now allowed due to full sandboxing enforcement on MHTML pages.
+ // MHTML pages can only be loaded from local URLs, http/https URLs, and
+ // content URLs(Android specific). The latter is now allowed due to full
+ // sandboxing enforcement on MHTML pages.
if (SchemeRegistry::shouldTreatURLSchemeAsLocal(url.protocol()))
return true;
if (url.protocolIsInHTTPFamily())
@@ -198,7 +200,8 @@ void MHTMLArchive::generateMHTMLPart(const String& boundary,
position += length;
}
} else {
- // FIXME: ideally we would encode the content as a stream without having to fetch it all.
+ // FIXME: ideally we would encode the content as a stream without having to
+ // fetch it all.
const char* data = resource.data->data();
size_t dataLength = resource.data->size();
Vector<char> encodedData;
@@ -208,7 +211,8 @@ void MHTMLArchive::generateMHTMLPart(const String& boundary,
outputBuffer.append("\r\n", 2u);
} else {
ASSERT(!strcmp(contentEncoding, base64));
- // We are not specifying insertLFs = true below as it would cut the lines with LFs and MHTML requires CRLFs.
+ // We are not specifying insertLFs = true below as it would cut the lines
+ // with LFs and MHTML requires CRLFs.
base64Encode(data, dataLength, encodedData);
const size_t maximumLineLength = 76;
size_t index = 0;
« no previous file with comments | « third_party/WebKit/Source/platform/mhtml/MHTMLArchive.h ('k') | third_party/WebKit/Source/platform/mhtml/MHTMLParser.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698