It's not clear from that if the IdentityCredential type defined happens to override that method (which is what's going on) or, that this specification defines a new implementation for Credential's DiscoverFromExternalSource, which is absolutely not what you intended.
Better wording is something along the lines of:
This specification introduces a new Credential interface {{IdentityCredential}}. {{IdentityCredential}} overrides the Credential's DiscoverFromExternalSource algorithm.
Another change is on the algorithm name & definition, the spec should call out this is IdentityCredential's DiscoverFromExternalSource that it's talking about:
Instead of
7.4. The [[DiscoverFromExternalSource]](origin, options, sameOriginWithAncestors) internal method
The spec says
It's not clear from that if the IdentityCredential type defined happens to override that method (which is what's going on) or, that this specification defines a new implementation for Credential's
DiscoverFromExternalSource, which is absolutely not what you intended.Better wording is something along the lines of:
Another change is on the algorithm name & definition, the spec should call out this is IdentityCredential's DiscoverFromExternalSource that it's talking about:
Instead of
Maybe