blink: Minor cleanups in core/script.

Change-Id: I348e13cf443af481479000d0b15874bb9a8f2dc0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2425931
Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Cr-Commit-Position: refs/heads/master@{#811193}

third_party/blink/renderer/core/script/script_loader.cc

@@ -976,10 +976,10 @@ void ScriptLoader::PendingScriptFinished(PendingScript* pending_script) {
              ScriptSchedulingType::kInOrder);
   // Historically we clear |resource_keep_alive_| when the scheduling type is
   // kAsync or kInOrder (crbug.com/778799). But if the script resource was
-  // served via signed exchange, the script may not be in the HTTPCache,
+  // served via signed exchange, the script may not be in the HTTPCache, and
   // therefore will need to be refetched over network if it's evicted from the
-  // memory cache not be in the HTTPCache. So we keep |resource_keep_alive_| to
-  // keep the resource in the memory cache.
+  // memory cache. So we keep |resource_keep_alive_| to keep the resource in the
+  // memory cache.
   if (resource_keep_alive_ &&
       !resource_keep_alive_->GetResponse().IsSignedExchangeInnerResponse() &&
       !base::FeatureList::IsEnabled(

third_party/blink/renderer/core/script/script_scheduling_type.h

@@ -9,14 +9,14 @@ namespace blink {
 
 // Type of <script>'s scheduling.
 //
-// In the spec, this is determined which clause of Step 25 of
-// https://html.spec.whatwg.org/C/#prepare-a-script
-// is taken.
+// <specdef href="https://html.spec.whatwg.org/C/#prepare-a-script">
+// This is determined by which clause of <spec step="28">Then, follow the first
+// of the following options that describes the situation:</spec> is taken.
 //
 // The enum values are used in histograms and thus do not change existing
 // enum values when modifying.
 enum class ScriptSchedulingType {
-  // Because the sheduling type is determined slightly after PendingScript
+  // Because the scheduling type is determined slightly after PendingScript
   // creation, it is set to kNotSet before ScriptLoader::TakePendingScript()
   // is called. kNotSet should not be observed.
   kNotSet,
@@ -66,14 +66,16 @@ enum class ScriptSchedulingType {
 
   // Force deferred scripts controlled by HTMLParserScriptRunner.
   // These are otherwise parser-blocking scripts that are being forced to
-  // execute after parsing completes (due to a ForceDeferScriptIntervention).
+  // execute after parsing completes (due to ForceDeferScriptIntervention).
   //
   // Spec: not yet spec'ed. https://crbug.com/976061
   kForceDefer
+
+  // When adding a new value, update kLastScriptSchedulingType.
 };
 
 static const int kLastScriptSchedulingType =
-    static_cast<int>(ScriptSchedulingType::kImmediate);
+    static_cast<int>(ScriptSchedulingType::kForceDefer);
 
 }  // namespace blink