Reflow comments in core/loader

third_party/WebKit/Source/core/loader/PingLoader.cpp

 /*
  * Copyright (C) 2010 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 130 matching lines @@
 
   unsigned long long size() const override { return m_data->byteLength(); }
 
   void serialize(ResourceRequest& request) const override {
     DCHECK(m_data);
 
     RefPtr<EncodedFormData> entityBody =
         EncodedFormData::create(m_data->baseAddress(), m_data->byteLength());
     request.setHTTPBody(entityBody.release());
 
-    // FIXME: a reasonable choice, but not in the spec; should it give a default?
+    // FIXME: a reasonable choice, but not in the spec; should it give a
+    // default?
     request.setHTTPContentType(AtomicString("application/octet-stream"));
   }
 
   const AtomicString getContentType() const { return nullAtom; }
 
  private:
   const Member<DOMArrayBufferView> m_data;
 };
 
 class BeaconFormData final : public Beacon {
@@ skipping 99 matching lines @@
   if (frame->frameScheduler())
     frame->frameScheduler()->didStopLoading(m_identifier);
 
   m_loader = wrapUnique(Platform::current()->createURLLoader());
   DCHECK(m_loader);
   WrappedResourceRequest wrappedRequest(request);
   wrappedRequest.setAllowStoredCredentials(credentialsAllowed ==
                                            AllowStoredCredentials);
   m_loader->loadAsynchronously(wrappedRequest, this);
 
-  // If the server never responds, FrameLoader won't be able to cancel this load and
-  // we'll sit here waiting forever. Set a very generous timeout, just in case.
+  // If the server never responds, FrameLoader won't be able to cancel this load
+  // and we'll sit here waiting forever. Set a very generous timeout, just in
+  // case.
   m_timeout.startOneShot(60000, BLINK_FROM_HERE);
 }
 
 PingLoaderImpl::~PingLoaderImpl() {
   if (m_loader)
     m_loader->cancel();
 }
 
 void PingLoaderImpl::dispose() {
   if (m_loader) {
     m_loader->cancel();
     m_loader = nullptr;
   }
   m_timeout.stop();
   m_keepAlive.clear();
 }
 
 void PingLoaderImpl::willFollowRedirect(
     WebURLLoader*,
     WebURLRequest& passedNewRequest,
     const WebURLResponse& passedRedirectResponse,
     int64_t encodedDataLength) {
   if (!m_isBeacon)
     return;
 
-  // TODO(tyoshino): Check if setAllowStoredCredentials() should be called
-  // also for non beacon cases.
+  // TODO(tyoshino): Check if setAllowStoredCredentials() should be called also
+  // for non beacon cases.
   passedNewRequest.setAllowStoredCredentials(true);
   if (m_corsMode == NotCORSEnabled)
     return;
 
   ResourceRequest& newRequest(passedNewRequest.toMutableResourceRequest());
   const ResourceResponse& redirectResponse(
       passedRedirectResponse.toResourceResponse());
 
   DCHECK(!newRequest.isNull());
   DCHECK(!redirectResponse.isNull());
 
   String errorDescription;
   ResourceLoaderOptions options;
   // TODO(tyoshino): Save updated data in options.securityOrigin and pass it
   // on the next time.
   if (!CrossOriginAccessControl::handleRedirect(
           m_origin, newRequest, redirectResponse, AllowStoredCredentials,
           options, errorDescription)) {
     if (LocalFrame* localFrame = frame()) {
       if (localFrame->document())
         localFrame->document()->addConsoleMessage(ConsoleMessage::create(
             JSMessageSource, ErrorMessageLevel, errorDescription));
     }
     // Cancel the load and self destruct.
     dispose();
     // Signal WebURLLoader that the redirect musn't be followed.
     passedNewRequest = WebURLRequest();
     return;
   }
-  // FIXME: http://crbug.com/427429 is needed to correctly propagate
-  // updates of Origin: following this successful redirect.
+  // FIXME: http://crbug.com/427429 is needed to correctly propagate updates of
+  // Origin: following this successful redirect.
 }
 
 void PingLoaderImpl::didReceiveResponse(WebURLLoader*,
                                         const WebURLResponse& response) {
   if (LocalFrame* frame = this->frame()) {
     TRACE_EVENT_INSTANT1(
         "devtools.timeline", "ResourceFinish", TRACE_EVENT_SCOPE_THREAD, "data",
         InspectorResourceFinishEvent::data(m_identifier, 0, true));
     const ResourceResponse& resourceResponse = response.toResourceResponse();
     InspectorInstrumentation::didReceiveResourceResponse(frame, m_identifier, 0,
@@ skipping 65 matching lines @@
 }
 
 bool sendPingCommon(LocalFrame* frame,
                     ResourceRequest& request,
                     const AtomicString& initiator,
                     StoredCredentials credentialsAllowed,
                     bool isBeacon) {
   if (MixedContentChecker::shouldBlockFetch(frame, request, request.url()))
     return false;
 
-  // The loader keeps itself alive until it receives a response and disposes itself.
+  // The loader keeps itself alive until it receives a response and disposes
+  // itself.
   PingLoaderImpl* loader = new PingLoaderImpl(frame, request, initiator,
                                               AllowStoredCredentials, true);
   DCHECK(loader);
 
   return true;
 }
 
 bool sendBeaconCommon(LocalFrame* frame,
                       int allowance,
                       const KURL& url,
@@ skipping 45 matching lines @@
     return;
 
   ResourceRequest request(pingURL);
   request.setHTTPMethod(HTTPNames::POST);
   request.setHTTPContentType("text/ping");
   request.setHTTPBody(EncodedFormData::create("PING"));
   request.setHTTPHeaderField(HTTPNames::Cache_Control, "max-age=0");
   finishPingRequestInitialization(request, frame,
                                   WebURLRequest::RequestContextPing);
 
-  // addAdditionalRequestHeaders() will have added a referrer for same origin requests,
-  // but the spec omits the referrer.
+  // addAdditionalRequestHeaders() will have added a referrer for same origin
+  // requests, but the spec omits the referrer.
   request.clearHTTPReferrer();
 
   request.setHTTPHeaderField(HTTPNames::Ping_To,
                              AtomicString(destinationURL.getString()));
 
   RefPtr<SecurityOrigin> pingOrigin = SecurityOrigin::create(pingURL);
   if (protocolIs(frame->document()->url().getString(), "http") ||
       frame->document()->getSecurityOrigin()->canAccess(pingOrigin.get()))
     request.setHTTPHeaderField(
         HTTPNames::Ping_From,
@@ skipping 55 matching lines @@
 bool PingLoader::sendBeacon(LocalFrame* frame,
                             int allowance,
                             const KURL& beaconURL,
                             Blob* data,
                             int& payloadLength) {
   BeaconBlob beacon(data);
   return sendBeaconCommon(frame, allowance, beaconURL, beacon, payloadLength);
 }
 
 }  // namespace blink