WebAuthn authenticatorAttachment is always "platform" over hybrid

[g-davidson]

f1464f7

WebAuthn authenticatorAttachment is always "platform" over hybrid
https://bugs.webkit.org/show_bug.cgi?id=248800
rdar://102608692

Reviewed by J Pascoe.

At the time this was written, the type of credential implied the attachment. Now
all of the credential types have an attachment property to use instead.

Canonical link: https://commits.webkit.org/257576@main

c704afa

Misc	iOS, tvOS & watchOS	macOS	Linux	Windows
✅ 🧪 style	✅ 🛠 ios	✅ 🛠 mac	✅ 🛠 wpe
	✅ 🛠 ios-sim	✅ 🛠 mac-AS-debug	✅ 🛠 gtk	✅ 🛠 wincairo
✅ 🧪 webkitperl	✅ 🧪 ios-wk2	✅ 🧪 api-mac	✅ 🧪 gtk-wk2
	✅ 🧪 api-ios	✅ 🧪 mac-wk1	✅ 🧪 api-gtk
	✅ 🛠 tv	❌ 🧪 mac-wk2
	✅ 🛠 tv-sim	❌ 🧪 mac-AS-debug-wk2
✅ 🛠 🧪 merge	✅ 🛠 watch	✅ 🧪 mac-wk2-stress
	✅ 🛠 watch-sim

[webkit-early-warning-system]

EWS run on current version of this PR (hash c704afa)

Misc	iOS, tvOS & watchOS	macOS	Linux	Windows
✅ 🧪 style	✅ 🛠 ios	✅ 🛠 mac	✅ 🛠 wpe
	✅ 🛠 ios-sim	✅ 🛠 mac-AS-debug	✅ 🛠 gtk	✅ 🛠 wincairo
✅ 🧪 webkitperl	✅ 🧪 ios-wk2	✅ 🧪 api-mac	✅ 🧪 gtk-wk2
	✅ 🧪 api-ios	✅ 🧪 mac-wk1	✅ 🧪 api-gtk
	✅ 🛠 tv	❌ 🧪 mac-wk2
	✅ 🛠 tv-sim	❌ 🧪 mac-AS-debug-wk2
✅ 🛠 🧪 merge	✅ 🛠 watch	✅ 🧪 mac-wk2-stress
	✅ 🛠 watch-sim

[rmondello]

@pascoej will likely be interested in this one.

[pascoej]

From the WK POV, I wish we could just have one of ASCPublicKeyCredentialRegistration.
r+

[pascoej]

@ryanhaddad mac-wk2 ran this both with and without this change and found the same test results, but the bubble is still red, is this a bug?

[webkit-commit-queue]

Committed 257576@main (f1464f7): https://commits.webkit.org/257576@main

Reviewed commits have been landed. Closing PR #7178 and removing active labels.