LibWeb: Don't trigger onchange event when setting <select> value

Author: https://github.com/Arhcout Commit: https://github.com/LadybirdBrowser/ladybird/commit/5d00211a860 Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/1522 Reviewed-by: https://github.com/gmta ✅
2024-09-25 10:56:02 +02:00 · 2024-09-25 10:56:02 +02:00 · 5d00211a86 · 2024-10-08 17:04:48 +00:00
commit 5d00211a86
parent f88acedc8f
3 changed files with 27 additions and 1 deletions
--- a/Tests/LibWeb/Text/expected/HTML/HTMLSelectElement-value-change-trigger-onchange.txt
+++ b/Tests/LibWeb/Text/expected/HTML/HTMLSelectElement-value-change-trigger-onchange.txt
@ -0,0 +1 @@
+  PASS
--- a/Tests/LibWeb/Text/input/HTML/HTMLSelectElement-value-change-trigger-onchange.html
+++ b/Tests/LibWeb/Text/input/HTML/HTMLSelectElement-value-change-trigger-onchange.html
@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<script src="../include.js"></script>
+<select id="select-test">
+    <option value="A"></option>
+    <option value="B"></option>
+</select>
+<script>
+    asyncTest(done => {
+        const selectElement = document.getElementById("select-test");
+        let onchangeTriggered = false;
+        
+        selectElement.onchange = () => {
+            onchangeTriggered = true;
+        }
+        
+        selectElement.value = "B";
+        setTimeout(() => {
+            if (!onchangeTriggered) {
+                println("PASS");
+            } else {
+                println("FAILED");
+            }
+            done();
+        }, 1);
+    });
+</script>
--- a/Userland/Libraries/LibWeb/HTML/HTMLSelectElement.cpp
+++ b/Userland/Libraries/LibWeb/HTML/HTMLSelectElement.cpp
@ -295,7 +295,6 @@ WebIDL::ExceptionOr<void> HTMLSelectElement::set_value(String const& value)
    for (auto const& option_element : list_of_options())
        option_element->set_selected(option_element->value() == value);
    update_inner_text_element();
-    queue_input_and_change_events();
    return {};
 }