From b8d683c5fb57fac120405d31792a363b6f4e0d85 Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Wed, 8 Sep 2021 19:52:09 +0200
Subject: [PATCH] LibWeb: Tweak for/event whitespace stripping in <script> to
 match spec

This is not a functional change, it merely reorders operations so that
we match spec language better.
---
 Userland/Libraries/LibWeb/HTML/HTMLScriptElement.cpp | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/Userland/Libraries/LibWeb/HTML/HTMLScriptElement.cpp b/Userland/Libraries/LibWeb/HTML/HTMLScriptElement.cpp
index d150f3832d..22375f2362 100644
--- a/Userland/Libraries/LibWeb/HTML/HTMLScriptElement.cpp
+++ b/Userland/Libraries/LibWeb/HTML/HTMLScriptElement.cpp
@@ -196,11 +196,14 @@ void HTMLScriptElement::prepare_script()
     // 16. If the script element has an event attribute and a for attribute, and the script's type is "classic", then:
     if (m_script_type == ScriptType::Classic && has_attribute(HTML::AttributeNames::event) && has_attribute(HTML::AttributeNames::for_)) {
         // 1. Let for be the value of the for attribute.
-        auto for_ = attribute(HTML::AttributeNames::for_).trim_whitespace();
-        // 2. Let event be the value of the event attribute.
-        auto event = attribute(HTML::AttributeNames::event).trim_whitespace();
+        auto for_ = attribute(HTML::AttributeNames::for_);
 
-        // FIXME: 3. Strip leading and trailing ASCII whitespace from event and for.
+        // 2. Let event be the value of the event attribute.
+        auto event = attribute(HTML::AttributeNames::event);
+
+        // 3. Strip leading and trailing ASCII whitespace from event and for.
+        for_ = for_.trim_whitespace();
+        event = event.trim_whitespace();
 
         // 4. If for is not an ASCII case-insensitive match for the string "window", then return. The script is not executed.
         if (!for_.equals_ignoring_case("window")) {