Reflow comments in core/fetch

third_party/WebKit/Source/core/fetch/ResourceLoaderOptions.h

 /*
  * 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 35 matching lines @@
   DoNotCheckContentSecurityPolicy
 };
 
 enum RequestInitiatorContext {
   DocumentContext,
   WorkerContext,
 };
 
 enum StoredCredentials { AllowStoredCredentials, DoNotAllowStoredCredentials };
 
-// APIs like XMLHttpRequest and EventSource let the user decide
-// whether to send credentials, but they're always sent for
-// same-origin requests. Additional information is needed to handle
-// cross-origin redirects correctly.
+// APIs like XMLHttpRequest and EventSource let the user decide whether to send
+// credentials, but they're always sent for same-origin requests. Additional
+// information is needed to handle cross-origin redirects correctly.
 enum CredentialRequest {
   ClientRequestedCredentials,
   ClientDidNotRequestCredentials
 };
 
 enum SynchronousPolicy { RequestSynchronously, RequestAsynchronously };
 
-// A resource fetch can be marked as being CORS enabled. The loader
-// must perform an access check upon seeing the response.
+// A resource fetch can be marked as being CORS enabled. The loader must perform
+// an access check upon seeing the response.
 enum CORSEnabled { NotCORSEnabled, IsCORSEnabled };
 
 struct ResourceLoaderOptions {
   USING_FAST_MALLOC(ResourceLoaderOptions);
 
  public:
   ResourceLoaderOptions()
       : dataBufferingPolicy(BufferData),
         allowCredentials(DoNotAllowStoredCredentials),
         credentialsRequested(ClientDidNotRequestCredentials),
         contentSecurityPolicyOption(CheckContentSecurityPolicy),
         requestInitiatorContext(DocumentContext),
         synchronousPolicy(RequestAsynchronously),
         corsEnabled(NotCORSEnabled) {}
 
   ResourceLoaderOptions(
       DataBufferingPolicy dataBufferingPolicy,
       StoredCredentials allowCredentials,
       CredentialRequest credentialsRequested,
       ContentSecurityPolicyDisposition contentSecurityPolicyOption,
       RequestInitiatorContext requestInitiatorContext)
       : dataBufferingPolicy(dataBufferingPolicy),
         allowCredentials(allowCredentials),
         credentialsRequested(credentialsRequested),
         contentSecurityPolicyOption(contentSecurityPolicyOption),
         requestInitiatorContext(requestInitiatorContext),
         synchronousPolicy(RequestAsynchronously),
         corsEnabled(NotCORSEnabled) {}
 
-  // Answers the question "can a separate request with these
-  // different options be re-used" (e.g. preload request)
-  // The safe (but possibly slow) answer is always false.
+  // Answers the question "can a separate request with these different options
+  // be re-used" (e.g. preload request) The safe (but possibly slow) answer is
+  // always false.
   bool canReuseRequest(const ResourceLoaderOptions& other) const {
     // dataBufferingPolicy differences are believed to be safe for re-use.
     // FIXME: check allowCredentials.
     // FIXME: check credentialsRequested.
     // FIXME: check contentSecurityPolicyOption.
     // initiatorInfo is purely informational and should be benign for re-use.
     // requestInitiatorContext is benign (indicates document vs. worker)
     // synchronousPolicy (safe to re-use an async XHR response for sync, etc.)
     return corsEnabled == other.corsEnabled;
-    // securityOrigin has more complicated checks which callers are responsible for.
+    // securityOrigin has more complicated checks which callers are responsible
+    // for.
   }
 
   // When adding members, CrossThreadResourceLoaderOptionsData should be
   // updated.
   DataBufferingPolicy dataBufferingPolicy;
-  StoredCredentials
-      allowCredentials;  // Whether HTTP credentials and cookies are sent with the request.
-  CredentialRequest
-      credentialsRequested;  // Whether the client (e.g. XHR) wanted credentials in the first place.
+
+  // Whether HTTP credentials and cookies are sent with the request.
+  StoredCredentials allowCredentials;
+
+  // Whether the client (e.g. XHR) wanted credentials in the first place.
+  CredentialRequest credentialsRequested;
+
   ContentSecurityPolicyDisposition contentSecurityPolicyOption;
   FetchInitiatorInfo initiatorInfo;
   RequestInitiatorContext requestInitiatorContext;
   SynchronousPolicy synchronousPolicy;
-  CORSEnabled
-      corsEnabled;  // If the resource is loaded out-of-origin, whether or not to use CORS.
+
+  // If the resource is loaded out-of-origin, whether or not to use CORS.
+  CORSEnabled corsEnabled;
+
   RefPtr<SecurityOrigin> securityOrigin;
   String contentSecurityPolicyNonce;
   IntegrityMetadataSet integrityMetadata;
 };
 
 // Encode AtomicString (in FetchInitiatorInfo) as String to cross threads.
 struct CrossThreadResourceLoaderOptionsData {
   DISALLOW_NEW();
   explicit CrossThreadResourceLoaderOptionsData(
       const ResourceLoaderOptions& options)
@@ skipping 44 matching lines @@
 struct CrossThreadCopier<ResourceLoaderOptions> {
   using Type = CrossThreadResourceLoaderOptionsData;
   static Type copy(const ResourceLoaderOptions& options) {
     return CrossThreadResourceLoaderOptionsData(options);
   }
 };
 
 }  // namespace blink
 
 #endif  // ResourceLoaderOptions_h