Skip to content

Commit

Permalink
Bug 1933655 [wpt PR 49391] - [devtools] Throw adding 2 internal authe…
Browse files Browse the repository at this point in the history
…nticators, a=testonly

Automatic update from web-platform-tests
[devtools] Throw adding 2 internal authenticators

Have devtools throw an error when a client attempts to add an internal
authenticator when there's one already present in the environment. This
is not supported and causes all sorts of issues.

This requires fixing a WPT that did not clean the virtual
authenticator. The WPT passing and not crashing Chrome previously was
luck.

Fixed: 40234612
Change-Id: I25d6e6c3387d0180234c21af157bc0b6ef15f566
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6047769
Reviewed-by: Alex Rudenko <[email protected]>
Auto-Submit: Nina Satragno <[email protected]>
Commit-Queue: Alex Rudenko <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1388668}

--

wpt-commits: d05c649bf1f54d0eec5e87d33b3ac7e8cdd1cc1b
wpt-pr: 49391
  • Loading branch information
nsatragno authored and moz-wptsync-bot committed Nov 28, 2024
1 parent cd1590b commit 30e9d7c
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

window.onload = async function() {
test_driver.set_test_context(parent);
await window.test_driver.add_virtual_authenticator(authenticatorArgs);
const authenticator = await window.test_driver.add_virtual_authenticator(authenticatorArgs);
let enabled = true;
let name = `OK`;
try {
Expand Down Expand Up @@ -56,6 +56,7 @@
enabled = false;
name = e.name;
}
await window.test_driver.remove_virtual_authenticator(authenticator);
parent.postMessage({ type: 'availability-result', enabled, name }, '*');
}
</script>

0 comments on commit 30e9d7c

Please sign in to comment.