From 627eb90086c70b150ab5a4acbd2d906657962cca Mon Sep 17 00:00:00 2001
From: Timothy Flynn <trflynn89@pm.me>
Date: Sat, 2 Nov 2024 11:19:55 -0400
Subject: [PATCH] LibWeb: Update WebDriver's list of collection types

HTMLAllCollection and HTMLFormControlsCollection were implemented since
this AO was written, and DOMTokenList was added to the spec.
---
 Userland/Libraries/LibWeb/WebDriver/ExecuteScript.cpp | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/Userland/Libraries/LibWeb/WebDriver/ExecuteScript.cpp b/Userland/Libraries/LibWeb/WebDriver/ExecuteScript.cpp
index c9d7726d818..c88cf279414 100644
--- a/Userland/Libraries/LibWeb/WebDriver/ExecuteScript.cpp
+++ b/Userland/Libraries/LibWeb/WebDriver/ExecuteScript.cpp
@@ -19,11 +19,14 @@
 #include <LibJS/Runtime/ObjectEnvironment.h>
 #include <LibJS/Runtime/Promise.h>
 #include <LibJS/Runtime/PromiseConstructor.h>
+#include <LibWeb/DOM/DOMTokenList.h>
 #include <LibWeb/DOM/Document.h>
 #include <LibWeb/DOM/HTMLCollection.h>
 #include <LibWeb/DOM/NodeList.h>
 #include <LibWeb/FileAPI/FileList.h>
 #include <LibWeb/HTML/BrowsingContext.h>
+#include <LibWeb/HTML/HTMLAllCollection.h>
+#include <LibWeb/HTML/HTMLFormControlsCollection.h>
 #include <LibWeb/HTML/HTMLOptionsCollection.h>
 #include <LibWeb/HTML/Scripting/Environments.h>
 #include <LibWeb/HTML/Scripting/TemporaryExecutionContext.h>
@@ -60,14 +63,16 @@ static bool is_collection(JS::Object const& value)
         value.has_parameter_map()
         // - instance of Array
         || is<JS::Array>(value)
+        // - instance of DOMTokenList
+        || is<DOM::DOMTokenList>(value)
         // - instance of FileList
         || is<FileAPI::FileList>(value)
         // - instance of HTMLAllCollection
-        || false // FIXME
+        || is<HTML::HTMLAllCollection>(value)
         // - instance of HTMLCollection
         || is<DOM::HTMLCollection>(value)
         // - instance of HTMLFormControlsCollection
-        || false // FIXME
+        || is<HTML::HTMLFormControlsCollection>(value)
         // - instance of HTMLOptionsCollection
         || is<HTML::HTMLOptionsCollection>(value)
         // - instance of NodeList