Reformat comments in core/html/parser

third_party/WebKit/Source/core/html/parser/HTMLPreloadScanner.cpp

 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
  * Copyright (C) 2009 Torch Mobile, Inc. http://www.torchmobile.com/
  * 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
@@ skipping 167 matching lines @@
       const Vector<CompactHTMLToken::Attribute>& attributes) {
     if (!m_tagImpl)
       return;
     for (const CompactHTMLToken::Attribute& htmlTokenAttribute : attributes)
       processAttribute(htmlTokenAttribute.name(), htmlTokenAttribute.value());
   }
 
   void handlePictureSourceURL(PictureData& pictureData) {
     if (match(m_tagImpl, sourceTag) && m_matched &&
         pictureData.sourceURL.isEmpty()) {
-      // Must create an isolatedCopy() since the srcset attribute value will
-      // get sent back to the main thread between when we set this, and when we
-      // process the closing tag which would clear m_pictureData. Having any
-      // ref to a string we're going to send will fail isSafeToSendToAnotherThread().
+      // Must create an isolatedCopy() since the srcset attribute value will get
+      // sent back to the main thread between when we set this, and when we
+      // process the closing tag which would clear m_pictureData. Having any ref
+      // to a string we're going to send will fail
+      // isSafeToSendToAnotherThread().
       pictureData.sourceURL = m_srcsetImageCandidate.toString().isolatedCopy();
       pictureData.sourceSizeSet = m_sourceSizeSet;
       pictureData.sourceSize = m_sourceSize;
       pictureData.picked = true;
     } else if (match(m_tagImpl, imgTag) && !pictureData.sourceURL.isEmpty()) {
       setUrlToLoad(pictureData.sourceURL, AllowURLReplacement);
     }
   }
 
   std::unique_ptr<PreloadRequest> createPreloadRequest(
@@ skipping 26 matching lines @@
     }
     if (sourceSizeSet) {
       resourceWidth.width = sourceSize;
       resourceWidth.isSet = true;
     }
 
     Resource::Type type;
     if (!resourceType(type))
       return nullptr;
 
-    // The element's 'referrerpolicy' attribute (if present) takes precedence over the document's referrer policy.
+    // The element's 'referrerpolicy' attribute (if present) takes precedence
+    // over the document's referrer policy.
     ReferrerPolicy referrerPolicy = (m_referrerPolicy != ReferrerPolicyDefault)
                                         ? m_referrerPolicy
                                         : documentReferrerPolicy;
     std::unique_ptr<PreloadRequest> request = PreloadRequest::create(
         initiatorFor(m_tagImpl), position, m_urlToLoad, predictedBaseURL, type,
         referrerPolicy, resourceWidth, clientHintsPreferences, requestType);
     request->setCrossOrigin(m_crossOrigin);
     request->setNonce(m_nonce);
     request->setCharset(charset());
     request->setDefer(m_defer);
     request->setIntegrityMetadata(m_integrityMetadata);
 
-    // TODO(csharrison): Once this is deprecated, just abort the request
-    // here.
+    // TODO(csharrison): Once this is deprecated, just abort the request here.
     if (match(m_tagImpl, scriptTag) &&
         !ScriptLoader::isValidScriptTypeAndLanguage(
             m_typeAttributeValue, m_languageAttributeValue,
             ScriptLoader::AllowLegacyTypeInTypeAttribute))
       request->setScriptHasInvalidTypeOrLanguage();
     return request;
   }
 
  private:
   template <typename NameType>
   void processScriptAttribute(const NameType& attributeName,
                               const String& attributeValue) {
     // FIXME - Don't set crossorigin multiple times.
     if (match(attributeName, srcAttr))
       setUrlToLoad(attributeValue, DisallowURLReplacement);
     else if (match(attributeName, crossoriginAttr))
       setCrossOrigin(attributeValue);
     else if (match(attributeName, nonceAttr))
       setNonce(attributeValue);
     else if (match(attributeName, asyncAttr))
       setDefer(FetchRequest::LazyLoad);
     else if (match(attributeName, deferAttr))
       setDefer(FetchRequest::LazyLoad);
     // Note that only scripts need to have the integrity metadata set on
     // preloads. This is because script resources fetches, and only script
-    // resource fetches, need to re-request resources if a cached version
-    // has different metadata (including empty) from the metadata on the
-    // request. See the comment before the call to
-    // mustRefetchDueToIntegrityMismatch() in
-    // Source/core/fetch/ResourceFetcher.cpp for a more complete
-    // explanation.
+    // resource fetches, need to re-request resources if a cached version has
+    // different metadata (including empty) from the metadata on the request.
+    // See the comment before the call to mustRefetchDueToIntegrityMismatch() in
+    // Source/core/fetch/ResourceFetcher.cpp for a more complete explanation.
     else if (match(attributeName, integrityAttr))
       SubresourceIntegrity::parseIntegrityAttribute(attributeValue,
                                                     m_integrityMetadata);
     else if (match(attributeName, typeAttr))
       m_typeAttributeValue = attributeValue;
     else if (match(attributeName, languageAttr))
       m_languageAttributeValue = attributeValue;
   }
 
   template <typename NameType>
@@ skipping 133 matching lines @@
     // http://www.whatwg.org/specs/web-apps/current-work/multipage/tokenization.html#attribute-name-state
     if (replacement == DisallowURLReplacement && !m_urlToLoad.isEmpty())
       return;
     String url = stripLeadingAndTrailingHTMLSpaces(value);
     if (url.isEmpty())
       return;
     m_urlToLoad = url;
   }
 
   const String& charset() const {
-    // FIXME: Its not clear that this if is needed, the loader probably ignores charset for image requests anyway.
+    // FIXME: Its not clear that this if is needed, the loader probably ignores
+    // charset for image requests anyway.
     if (match(m_tagImpl, imgTag) || match(m_tagImpl, videoTag))
       return emptyString();
     return m_charset;
   }
 
   bool resourceType(Resource::Type& type) const {
     if (match(m_tagImpl, scriptTag)) {
       type = Resource::Script;
     } else if (match(m_tagImpl, imgTag) || match(m_tagImpl, videoTag) ||
                (match(m_tagImpl, inputTag) && m_inputIsImage)) {
@@ skipping 94 matching lines @@
 
 TokenPreloadScannerCheckpoint TokenPreloadScanner::createCheckpoint() {
   TokenPreloadScannerCheckpoint checkpoint = m_checkpoints.size();
   m_checkpoints.append(Checkpoint(m_predictedBaseElementURL, m_inStyle,
                                   m_inScript, m_templateCount));
   return checkpoint;
 }
 
 void TokenPreloadScanner::rewindTo(
     TokenPreloadScannerCheckpoint checkpointIndex) {
-  ASSERT(checkpointIndex <
-         m_checkpoints
-             .size());  // If this ASSERT fires, checkpointIndex is invalid.
+  // If this ASSERT fires, checkpointIndex is invalid.
+  ASSERT(checkpointIndex < m_checkpoints.size());
   const Checkpoint& checkpoint = m_checkpoints[checkpointIndex];
   m_predictedBaseElementURL = checkpoint.predictedBaseElementURL;
   m_inStyle = checkpoint.inStyle;
   m_templateCount = checkpoint.templateCount;
 
   m_didRewind = true;
   m_inScript = checkpoint.inScript;
 
   m_cssScanner.reset();
   m_checkpoints.clear();
@@ skipping 153 matching lines @@
 
   switch (token.type()) {
     case HTMLToken::Character: {
       if (m_inStyle) {
         m_cssScanner.scan(token.data(), source, requests,
                           m_predictedBaseElementURL);
       } else if (m_inScript && likelyDocumentWriteScript && !m_didRewind) {
         // Don't mark scripts for evaluation if the preloader rewound to a
         // previous checkpoint. This could cause re-evaluation of scripts if
         // care isn't given.
-        // TODO(csharrison): Revisit this if rewinds are low hanging fruit for the
-        // document.write evaluator.
+        // TODO(csharrison): Revisit this if rewinds are low hanging fruit for
+        // the document.write evaluator.
         *likelyDocumentWriteScript =
             shouldEvaluateForDocumentWrite(token.data());
       }
       return;
     }
     case HTMLToken::EndTag: {
       const StringImpl* tagImpl = tagImplFor(token.data());
       if (match(tagImpl, templateTag)) {
         if (m_templateCount)
           --m_templateCount;
@@ skipping 18 matching lines @@
         return;
       const StringImpl* tagImpl = tagImplFor(token.data());
       if (match(tagImpl, templateTag)) {
         ++m_templateCount;
         return;
       }
       if (match(tagImpl, styleTag)) {
         m_inStyle = true;
         return;
       }
-      // Don't early return, because the StartTagScanner needs to look at
-      // these too.
+      // Don't early return, because the StartTagScanner needs to look at these
+      // too.
       if (match(tagImpl, scriptTag)) {
         m_inScript = true;
       }
       if (match(tagImpl, baseTag)) {
         // The first <base> element is the one that wins.
         if (!m_predictedBaseElementURL.isEmpty())
           return;
         updatePredictedBaseURL(token);
         return;
       }
@@ skipping 63 matching lines @@
 
 HTMLPreloadScanner::~HTMLPreloadScanner() {}
 
 void HTMLPreloadScanner::appendToEnd(const SegmentedString& source) {
   m_source.append(source);
 }
 
 void HTMLPreloadScanner::scanAndPreload(ResourcePreloader* preloader,
                                         const KURL& startingBaseElementURL,
                                         ViewportDescriptionWrapper* viewport) {
-  ASSERT(
-      isMainThread());  // HTMLTokenizer::updateStateFor only works on the main thread.
+  // HTMLTokenizer::updateStateFor only works on the main thread.
+  ASSERT(isMainThread());
 
   TRACE_EVENT1("blink", "HTMLPreloadScanner::scan", "source_length",
                m_source.length());
 
-  // When we start scanning, our best prediction of the baseElementURL is the real one!
+  // When we start scanning, our best prediction of the baseElementURL is the
+  // real one!
   if (!startingBaseElementURL.isEmpty())
     m_scanner.setPredictedBaseElementURL(startingBaseElementURL);
 
   PreloadRequestStream requests;
 
   while (m_tokenizer->nextToken(m_source, m_token)) {
     if (m_token.type() == HTMLToken::StartTag)
       m_tokenizer->updateStateFor(
           attemptStaticStringCreation(m_token.name(), Likely8Bit));
     bool isCSPMetaTag = false;
     m_scanner.scan(m_token, m_source, requests, viewport, &isCSPMetaTag);
     m_token.clear();
-    // Don't preload anything if a CSP meta tag is found. We should never
-    // really find them here because the HTMLPreloadScanner is only used for
+    // Don't preload anything if a CSP meta tag is found. We should never really
+    // find them here because the HTMLPreloadScanner is only used for
     // dynamically added markup.
     if (isCSPMetaTag)
       return;
   }
 
   preloader->takeAndPreload(requests);
 }
 
 CachedDocumentParameters::CachedDocumentParameters(Document* document) {
   ASSERT(isMainThread());
   ASSERT(document);
   doHtmlPreloadScanning =
       !document->settings() || document->settings()->doHtmlPreloadScanning();
   doDocumentWritePreloadScanning = doHtmlPreloadScanning && document->frame() &&
                                    document->frame()->isMainFrame();
   defaultViewportMinWidth = document->viewportDefaultMinWidth();
   viewportMetaZeroValuesQuirk =
       document->settings() &&
       document->settings()->viewportMetaZeroValuesQuirk();
   viewportMetaEnabled =
       document->settings() && document->settings()->viewportMetaEnabled();
   referrerPolicy = document->getReferrerPolicy();
 }
 
 }  // namespace blink