Reflow comments in core/dom/.

third_party/WebKit/Source/core/dom/ScriptLoader.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
- * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
+ * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights
+ * reserved.
  * Copyright (C) 2008 Nikolas Zimmermann <zimmermann@kde.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
@@ skipping 107 matching lines @@
 
 void ScriptLoader::detach() {
   if (!m_pendingScript)
     return;
   m_pendingScript->dispose();
   m_pendingScript = nullptr;
 }
 
 // Helper function. Must take a lowercase language as input.
 static bool isLegacySupportedJavaScriptLanguage(const String& language) {
-  // Mozilla 1.8 accepts javascript1.0 - javascript1.7, but WinIE 7 accepts only javascript1.1 - javascript1.3.
+  // Mozilla 1.8 accepts javascript1.0 - javascript1.7, but WinIE 7 accepts only
+  // javascript1.1 - javascript1.3.
   // Mozilla 1.8 and WinIE 7 both accept javascript and livescript.
   // WinIE 7 accepts ecmascript and jscript, but Mozilla 1.8 doesn't.
   // Neither Mozilla 1.8 nor WinIE 7 accept leading or trailing whitespace.
-  // We want to accept all the values that either of these browsers accept, but not other values.
+  // We want to accept all the values that either of these browsers accept, but
+  // not other values.
 
-  // FIXME: This function is not HTML5 compliant. These belong in the MIME registry as "text/javascript<version>" entries.
+  // FIXME: This function is not HTML5 compliant. These belong in the MIME
+  // registry as "text/javascript<version>" entries.
   DCHECK_EQ(language, language.lower());
   return language == "javascript" || language == "javascript1.0" ||
          language == "javascript1.1" || language == "javascript1.2" ||
          language == "javascript1.3" || language == "javascript1.4" ||
          language == "javascript1.5" || language == "javascript1.6" ||
          language == "javascript1.7" || language == "livescript" ||
          language == "ecmascript" || language == "jscript";
 }
 
 void ScriptLoader::dispatchErrorEvent() {
   m_element->dispatchEvent(Event::create(EventTypeNames::error));
 }
 
 void ScriptLoader::dispatchLoadEvent() {
   if (ScriptLoaderClient* client = this->client())
     client->dispatchLoadEvent();
   setHaveFiredLoadEvent(true);
 }
 
 bool ScriptLoader::isValidScriptTypeAndLanguage(
     const String& type,
     const String& language,
     LegacyTypeSupport supportLegacyTypes) {
-  // FIXME: isLegacySupportedJavaScriptLanguage() is not valid HTML5. It is used here to maintain backwards compatibility with existing layout tests. The specific violations are:
-  // - Allowing type=javascript. type= should only support MIME types, such as text/javascript.
-  // - Allowing a different set of languages for language= and type=. language= supports Javascript 1.1 and 1.4-1.6, but type= does not.
+  // FIXME: isLegacySupportedJavaScriptLanguage() is not valid HTML5. It is used
+  // here to maintain backwards compatibility with existing layout tests. The
+  // specific violations are:
+  // - Allowing type=javascript. type= should only support MIME types, such as
+  //   text/javascript.
+  // - Allowing a different set of languages for language= and type=. language=
+  //   supports Javascript 1.1 and 1.4-1.6, but type= does not.
   if (type.isEmpty()) {
     String lowerLanguage = language.lower();
     return language.isEmpty()  // assume text/javascript.
            || MIMETypeRegistry::isSupportedJavaScriptMIMEType("text/" +
                                                               lowerLanguage) ||
            isLegacySupportedJavaScriptLanguage(lowerLanguage);
   } else if (RuntimeEnabledFeatures::moduleScriptsEnabled() &&
              type == "module") {
     return true;
   } else if (MIMETypeRegistry::isSupportedJavaScriptMIMEType(
@@ skipping 25 matching lines @@
   if (m_parserInserted) {
     wasParserInserted = true;
     m_parserInserted = false;
   } else {
     wasParserInserted = false;
   }
 
   if (wasParserInserted && !client->asyncAttributeValue())
     m_forceAsync = true;
 
-  // FIXME: HTML5 spec says we should check that all children are either comments or empty text nodes.
+  // FIXME: HTML5 spec says we should check that all children are either
+  // comments or empty text nodes.
   if (!client->hasSourceAttribute() && !m_element->hasChildren())
     return false;
 
   if (!m_element->isConnected())
     return false;
 
   if (!isScriptTypeSupported(supportLegacyTypes))
     return false;
 
   if (wasParserInserted) {
     m_parserInserted = true;
     m_forceAsync = false;
   }
 
   m_alreadyStarted = true;
 
-  // FIXME: If script is parser inserted, verify it's still in the original document.
+  // FIXME: If script is parser inserted, verify it's still in the original
+  // document.
   Document& elementDocument = m_element->document();
   Document* contextDocument = elementDocument.contextDocument();
 
   if (!contextDocument || !contextDocument->allowExecutingScripts(m_element))
     return false;
 
   if (!isScriptForEventSupported())
     return false;
 
   if (!client->charsetAttributeValue().isEmpty())
@@ skipping 89 matching lines @@
         ResourceRequest(elementDocument->completeURL(sourceUrl)),
         m_element->localName());
 
     CrossOriginAttributeValue crossOrigin = crossOriginAttributeValue(
         m_element->fastGetAttribute(HTMLNames::crossoriginAttr));
     if (crossOrigin != CrossOriginAttributeNotSet)
       request.setCrossOriginAccessControl(elementDocument->getSecurityOrigin(),
                                           crossOrigin);
     request.setCharset(scriptCharset());
 
-    // Skip fetch-related CSP checks if dynamically injected script is whitelisted and this script is not parser-inserted.
+    // Skip fetch-related CSP checks if dynamically injected script is
+    // whitelisted and this script is not parser-inserted.
     bool scriptPassesCSPDynamic =
         (!isParserInserted() &&
          elementDocument->contentSecurityPolicy()->allowDynamic());
 
     if (ContentSecurityPolicy::isNonceableElement(m_element.get()))
       request.setContentSecurityPolicyNonce(
           m_element->fastGetAttribute(HTMLNames::nonceAttr));
 
     if (scriptPassesCSPDynamic) {
       UseCounter::count(elementDocument->frame(),
@@ skipping 173 matching lines @@
         accessControlStatus = OpaqueResource;
       else
         accessControlStatus = SharableCrossOrigin;
     } else if (sourceCode.resource()->passesAccessControlCheck(
                    m_element->document().getSecurityOrigin())) {
       accessControlStatus = SharableCrossOrigin;
     }
   }
 
   const bool isImportedScript = contextDocument != elementDocument;
-  // http://www.whatwg.org/specs/web-apps/current-work/#execute-the-script-block step 2.3
-  // with additional support for HTML imports.
+  // http://www.whatwg.org/specs/web-apps/current-work/#execute-the-script-block
+  // step 2.3 with additional support for HTML imports.
   IgnoreDestructiveWriteCountIncrementer ignoreDestructiveWriteCountIncrementer(
       m_isExternalScript || isImportedScript ? contextDocument : 0);
 
   if (isHTMLScriptLoader(m_element) || isSVGScriptLoader(m_element))
     contextDocument->pushCurrentScript(m_element);
 
   // Create a script from the script element node, using the script
   // block's source and the script block's type.
   // Note: This is where the script is compiled and actually executed.
   frame->script().executeScriptInMainWorld(sourceCode, accessControlStatus);
@@ skipping 98 matching lines @@
   if (isHTMLScriptLoader(element))
     return toHTMLScriptElement(element)->loader();
 
   if (isSVGScriptLoader(element))
     return toSVGScriptElement(element)->loader();
 
   return 0;
 }
 
 }  // namespace blink