Reflow comments in core/fetch

third_party/WebKit/Source/core/fetch/RawResource.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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 165 matching lines @@
 void RawResource::responseReceived(
     const ResourceResponse& response,
     std::unique_ptr<WebDataConsumerHandle> handle) {
   bool isSuccessfulRevalidation =
       isCacheValidator() && response.httpStatusCode() == 304;
   Resource::responseReceived(response, nullptr);
 
   ResourceClientWalker<RawResourceClient> w(clients());
   DCHECK(clients().size() <= 1 || !handle);
   while (RawResourceClient* c = w.next()) {
-    // |handle| is cleared when passed, but it's not a problem because
-    // |handle| is null when there are two or more clients, as asserted.
+    // |handle| is cleared when passed, but it's not a problem because |handle|
+    // is null when there are two or more clients, as asserted.
     c->responseReceived(this, this->response(), std::move(handle));
   }
 
-  // If we successfully revalidated, we won't get appendData() calls.
-  // Forward the data to clients now instead.
-  // Note: |m_data| can be null when no data is appended to the original
-  // resource.
+  // If we successfully revalidated, we won't get appendData() calls. Forward
+  // the data to clients now instead. Note: |m_data| can be null when no data is
+  // appended to the original resource.
   if (isSuccessfulRevalidation && data()) {
     ResourceClientWalker<RawResourceClient> w(clients());
     while (RawResourceClient* c = w.next())
       c->dataReceived(this, data()->data(), data()->size());
   }
 }
 
 void RawResource::setSerializedCachedMetadata(const char* data, size_t size) {
   Resource::setSerializedCachedMetadata(data, size);
   ResourceClientWalker<RawResourceClient> w(clients());
@@ skipping 20 matching lines @@
   while (RawResourceClient* c = w.next())
     c->didReceiveResourceTiming(this, info);
 }
 
 void RawResource::setDefersLoading(bool defers) {
   if (loader())
     loader()->setDefersLoading(defers);
 }
 
 static bool shouldIgnoreHeaderForCacheReuse(AtomicString headerName) {
-  // FIXME: This list of headers that don't affect cache policy almost certainly isn't complete.
+  // FIXME: This list of headers that don't affect cache policy almost certainly
+  // isn't complete.
   DEFINE_STATIC_LOCAL(
       HashSet<AtomicString>, headers,
       ({
           "Cache-Control", "If-Modified-Since", "If-None-Match", "Origin",
           "Pragma", "Purpose", "Referer", "User-Agent",
           HTTPNames::X_DevTools_Emulate_Network_Conditions_Client_Id,
       }));
   return headers.contains(headerName);
 }
 
@@ skipping 12 matching lines @@
   if (resourceRequest().httpMethod() != newRequest.httpMethod())
     return false;
 
   if (resourceRequest().httpBody() != newRequest.httpBody())
     return false;
 
   if (resourceRequest().allowStoredCredentials() !=
       newRequest.allowStoredCredentials())
     return false;
 
-  // Ensure most headers match the existing headers before continuing.
-  // Note that the list of ignored headers includes some headers explicitly related to caching.
-  // A more detailed check of caching policy will be performed later, this is simply a list of
-  // headers that we might permit to be different and still reuse the existing Resource.
+  // Ensure most headers match the existing headers before continuing. Note that
+  // the list of ignored headers includes some headers explicitly related to
+  // caching. A more detailed check of caching policy will be performed later,
+  // this is simply a list of headers that we might permit to be different and
+  // still reuse the existing Resource.
   const HTTPHeaderMap& newHeaders = newRequest.httpHeaderFields();
   const HTTPHeaderMap& oldHeaders = resourceRequest().httpHeaderFields();
 
   for (const auto& header : newHeaders) {
     AtomicString headerName = header.key;
     if (!shouldIgnoreHeaderForCacheReuse(headerName) &&
         header.value != oldHeaders.get(headerName))
       return false;
   }
 
@@ skipping 73 matching lines @@
   SECURITY_CHECK(m_state != NotAddedAsClient);
   SECURITY_CHECK(m_state != NotifyFinished);
   SECURITY_CHECK(resource->errorOccurred() ||
                  (m_state == ResponseReceived ||
                   m_state == SetSerializedCachedMetadata ||
                   m_state == DataReceived || m_state == DataDownloaded));
   m_state = NotifyFinished;
 }
 
 }  // namespace blink