Fix module scripts with for/event attributes

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) 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
@@ skipping 946 matching lines @@
 
 // Step 13 of https://html.spec.whatwg.org/#prepare-a-script
 bool ScriptLoader::IsScriptForEventSupported() const {
   // 1. "Let for be the value of the for attribute."
   String event_attribute = element_->EventAttributeValue();
   // 2. "Let event be the value of the event attribute."
   String for_attribute = element_->ForAttributeValue();
 
   // "If the script element has an event attribute and a for attribute, and
   //  the script's type is "classic", then run these substeps:"
-  // TODO(hiroshige): Check the script's type.
-  if (event_attribute.IsNull() || for_attribute.IsNull())
+  if (GetScriptType() != ScriptType::kClassic || event_attribute.IsNull() ||
+      for_attribute.IsNull())
     return true;
 
   // 3. "Strip leading and trailing ASCII whitespace from event and for."
   for_attribute = for_attribute.StripWhiteSpace();
   // 4. "If for is not an ASCII case-insensitive match for the string
   //     "window",
   //     then abort these steps at this point. The script is not executed."
   if (!DeprecatedEqualIgnoringCase(for_attribute, "window"))
     return false;
   event_attribute = event_attribute.StripWhiteSpace();
   // 5. "If event is not an ASCII case-insensitive match for either the
   //     string "onload" or the string "onload()",
   //     then abort these steps at this point. The script is not executed.
   return DeprecatedEqualIgnoringCase(event_attribute, "onload") ||
          DeprecatedEqualIgnoringCase(event_attribute, "onload()");
 }
 
 String ScriptLoader::ScriptContent() const {
   return element_->TextFromChildren();
 }
 
 }  // namespace blink