tree f3c1186a3ed8567b8bad297bf7ed0fa72fc3d2e7
parent a0ece3bbdbfdf1bd0f66f841b06d54c1d927af0a
author Mason Freed <masonf@chromium.org> 1661208228 -0700
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1661208952 -0700

Only focus previously focused element if focus moves on showPopUp()

This fixes a bug wherein focus never moves when the pop-up is shown,
yet on hidePopUp(), focus is still restored. This is both confusing
and error-prone. See crbug.com/1354293 for an example.

Also remove the `autofocus_only` parameter from
`GetPopupFocusableArea()`, which was only ever set to true.

Bug: 1354293,1307772
Change-Id: Ie26819e28b28cb08c5168b78ac2b0834f643a32e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3842639
Auto-Submit: Mason Freed <masonf@chromium.org>
Commit-Queue: Mason Freed <masonf@chromium.org>
Reviewed-by: David Baron <dbaron@chromium.org>
Commit-Queue: David Baron <dbaron@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1037995}