Bug 1928486 [wpt PR 48913] - Make <selectedoption> stop responding to…

… mutations, a=testonly

Automatic update from web-platform-tests
Make <selectedoption> stop responding to mutations

This patch makes <selectedoption> stop using the MutationObserver for
<option>: openui/open-ui#825

This patch also uncovered a small bug with the
selectedoptionelement attribute when assigning null to the IDL
attribute, which I fixed by removing a null check.

Bug: 336844298
Change-Id: I3fed171bf687cabb9474d8ff8741071c28bbad0b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5941575
Reviewed-by: Dominic Farolino <[email protected]>
Commit-Queue: Joey Arhar <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1376573}

--

wpt-commits: ff425b2cbd0763d18844cf91cde9da9c8616f076
wpt-pr: 48913

testing/web-platform/tests/html/semantics/forms/the-select-element/customizable-select/selectedoption.tentative.html

@@ -44,20 +44,21 @@
   assert_equals(selectedOption.innerHTML, optionTwo.innerHTML,
     'The innerHTML of <selectedoption> should change after the selected option is changed.');
 
+  let oldInnerHTML = optionTwo.innerHTML;
   spanTwo.textContent = 'new span';
   await new Promise(queueMicrotask);
-  assert_equals(selectedOption.innerHTML, optionTwo.innerHTML,
-    '<selectedoption> should respond to text content changes.');
+  assert_equals(selectedOption.innerHTML, oldInnerHTML,
+    '<selectedoption> should not respond to <option> text content changes.');
 
   spanTwo.appendChild(document.createElement('div'));
   await new Promise(queueMicrotask);
-  assert_equals(selectedOption.innerHTML, optionTwo.innerHTML,
-    '<selectedoption> should respond to new elements being added to descendants.');
+  assert_equals(selectedOption.innerHTML, oldInnerHTML,
+    '<selectedoption> should not respond to new elements being added to descendants of <option>.');
 
   spanTwo.setAttribute('data-foo', 'bar');
   await new Promise(queueMicrotask);
-  assert_equals(selectedOption.innerHTML, optionTwo.innerHTML,
-    '<selectedoption> should respond to attributes being added to descendants.');
+  assert_equals(selectedOption.innerHTML, oldInnerHTML,
+    '<selectedoption> should not respond to attributes being added to descendants of <option>.');
 
   form.reset();
   await new Promise(queueMicrotask);
@@ -90,5 +91,7 @@
   optionOne.remove();
   assert_equals(selectedOption.innerHTML, '',
     'The content of <selectedoption> should be cleared if there is no selected <option>.');
+
+  // TODO(crbug.com/336844298): Add tests for mutation records during parsing
 }, 'The <selectedoption> element should reflect the HTML contents of the selected <option>.');
 </script>

testing/web-platform/tests/html/semantics/forms/the-select-element/customizable-select/selectedoptionelement-attr.tentative.html

@@ -34,11 +34,14 @@
     'Removing the selectedoptionelement attribute via IDL should synchronously clear the contents of the <selectedoption>.');
 
   select.selectedOptionElement = selectedoption;
+  assert_equals(selectedoption.innerHTML, optionOne.innerHTML,
+    'Re-setting the selectedoptionelement attribute via IDL should synchronously assign the contents of <selectedoption>.');
 
+  let oldInnerHTML = optionOne.innerHTML;
   optionOne.querySelector('span').remove();
   await new Promise(queueMicrotask);
-  assert_equals(selectedoption.innerHTML, optionOne.innerHTML,
-    'Mutating the selected <option> should update the <selectedoption> contents after a microtask.');
+  assert_equals(selectedoption.innerHTML, oldInnerHTML,
+    'Mutating the selected <option> should not update the <selectedoption> contents.');
 
   select.value = 'two';
   assert_equals(selectedoption.innerHTML, optionTwo.innerHTML,