Remove focus fixup for removal of <dialog open> attribute

See the conversation here:
  whatwg/html#10953 (comment)

This changes the behavior of the DialogCloseWhenOpenRemovedEnabled flag
just a bit, to move toward "option 2" from the comment just below the
one linked above. It also cleans up the two parameters to ::close(),
and replaces them with one. And it stops doing the focus fixup
behaviors in that case.

Bug: 376516550
Change-Id: I072d710df47bbe58f5137070591378c72f8124b2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6244982
Auto-Submit: Mason Freed <[email protected]>
Commit-Queue: Joey Arhar <[email protected]>
Reviewed-by: Joey Arhar <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1419540}

Author: mfreed7

html/semantics/interactive-elements/the-dialog-element/dialog-close-via-attribute.tentative.html

@@ -33,8 +33,8 @@
   await new Promise(resolve => t.step_timeout(resolve, 0));
   await new Promise(requestAnimationFrame);
 
-  assert_equals(document.activeElement, button,
-    'Previously focused element should be focused after waiting for a task.');
+  assert_not_equals(document.activeElement, button,
+    'Previously focused element should not be focused, even after waiting for a task.');
   assert_false(dialog.matches(':modal'),
     'The dialog should not match :modal after closing.');
   assert_false(cancelFired,
@@ -47,6 +47,9 @@
   await test_driver.click(button);
   assert_true(buttonFiredClick,
     'The page should not be inert or blocked after removing the open attribute.');
+  // Clean up
+  dialog.showModal();
+  dialog.close();
 }, 'Removing the open attribute from an open modal dialog should run the closing algorithm.');
 
 promise_test(async t => {
@@ -66,11 +69,14 @@
   await new Promise(resolve => t.step_timeout(resolve, 0));
   await new Promise(requestAnimationFrame);
 
-  assert_equals(document.activeElement, button,
-    'Previously focused element should be focused after waiting for a task.');
+  assert_not_equals(document.activeElement, button,
+    'Previously focused element should not be focused, even after waiting for a task.');
   assert_false(cancelFired,
     'The cancel event should not fire when removing the open attribute.');
   assert_true(closeFired,
     'The close event should be fired when removing the open attribute.');
+  // Clean up
+  dialog.show();
+  dialog.close();
 }, 'Removing the open attribute from an open non-modal dialog should fire a close event.');
 </script>