Fix the end index in setRangeText() for input and textarea

Also fixes the timing of the select event firing to be from a queued task, instead of synchronous.

Closes #2601.

lib/jsdom/living/nodes/HTMLInputElement-impl.js

@@ -596,7 +596,7 @@ class HTMLInputElementImpl extends HTMLElementImpl {
   }
 
   _dispatchSelectEvent() {
-    fireAnEvent("select", this, undefined, { bubbles: true, cancelable: true });
+    setTimeout(() => fireAnEvent("select", this, undefined, { bubbles: true, cancelable: false }), 0);
   }
 
   _getValueLength() {
@@ -694,7 +694,7 @@ class HTMLInputElementImpl extends HTMLElementImpl {
 
     this.value = val.slice(0, start) + repl + val.slice(end);
 
-    const newEnd = start + this.value.length;
+    const newEnd = start + repl.length;
 
     if (selectionMode === "select") {
       this.setSelectionRange(start, newEnd);

lib/jsdom/living/nodes/HTMLTextAreaElement-impl.js

@@ -94,7 +94,7 @@ class HTMLTextAreaElementImpl extends HTMLElementImpl {
   }
 
   _dispatchSelectEvent() {
-    fireAnEvent("select", this, undefined, { bubbles: true, cancelable: true });
+    setTimeout(() => fireAnEvent("select", this, undefined, { bubbles: true, cancelable: false }), 0);
   }
 
   _getValueLength() {
@@ -156,7 +156,7 @@ class HTMLTextAreaElementImpl extends HTMLElementImpl {
 
     this.value = val.slice(0, start) + repl + val.slice(end);
 
-    const newEnd = start + this.value.length;
+    const newEnd = start + repl.length;
 
     if (selectionMode === "select") {
       this.setSelectionRange(start, newEnd);

test/web-platform-tests/to-run.yaml

@@ -902,8 +902,6 @@ DIR: html/semantics/forms/textfieldselection
 
 select-event.html: [fail, not yet implemented]
 selection-start-end-extra.html: [fail, not yet implemented. Shortening value by turning input type into url - fix would cause the-input-element type-change-state.html to fail.]
-textfieldselection-setRangeText.html: [fail-slow, not yet implemented]
-textfieldselection-setSelectionRange.html: [fail, not yet implemented]
 
 ---
 

test/web-platform-tests/to-upstream/html/semantics/forms/the-textarea-element/setRangeText.html

@@ -37,7 +37,7 @@
 
     assert_equals(element.value, "bazbaz");
     assert_equals(element.selectionStart, 0);
-    assert_equals(element.selectionEnd, 6);
+    assert_equals(element.selectionEnd, 3);
 
   }, "textarea setRangeText(text, start, end, 'select') should create a new selection using start and end as bounds");
 
@@ -60,8 +60,8 @@
     element.setRangeText("baz", 0, 6, "end");
 
     assert_equals(element.value, "bazbaz");
-    assert_equals(element.selectionStart, 6);
-    assert_equals(element.selectionEnd, 6);
+    assert_equals(element.selectionStart, 3);
+    assert_equals(element.selectionEnd, 3);
 
   }, "textarea setRangeText(text, start, end, 'end') should create a new selection, collapsed to end");