Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Ted Thibodeau Jr <[email protected]>
  • Loading branch information
npm1 and TallTed authored Dec 1, 2023
1 parent db413f2 commit 8fa69b4
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions spec/index.bs
Original file line number Diff line number Diff line change
Expand Up @@ -472,7 +472,7 @@ To <dfn>compute the connection status</dfn> given an {{IdentityProviderConfig}}
1. Let |triple| be (|rpOrigin|, |idpOrigin|, |accountId|).
1. If [=connected accounts set=] [=list/contains=] |triple| , return
[=compute the connection status/connected=].
1. Otherwise, if account| [=map/contains=] {{IdentityProviderAccount/approved_clients}},
1. Otherwise, if |account| [=map/contains=] {{IdentityProviderAccount/approved_clients}},
|account|'s {{IdentityProviderAccount/approved_clients}} [=list/contains=] |provider|'s
{{IdentityProviderConfig/clientId}}, and third party cookies are available for |idpOrigin|
on |rpOrigin|, return [=compute the connection status/connected=].
Expand Down Expand Up @@ -1421,9 +1421,9 @@ When invoking the {{IdentityProvider/getUserInfo()}} method given an {{IdentityP
* There does not exist an account |account| such that [=compute the connection status=] of
|provider|, |account|, and |globalObject| returns
[=compute the connection status/connected=]. This check can be performed by iterating
over the [=connected accounts set=] or by keeping a separate data structure to make this
lookup fast.
* There is third party cookie access for |configUrl|'s [=url/origin=] on |document|'s
over the [=connected accounts set=] or by keeping a separate data structure for faster
lookup results.
* There is no third party cookie access for |configUrl|'s [=url/origin=] on |document|'s
[=Document/origin=].
1. If |configUrl| is failure, throw an "{{InvalidStateError}}" {{DOMException}}.
1. If |document|'s [=Document/origin=] is not [=same origin=] as |configUrl|'s [=url/origin=],
Expand Down

0 comments on commit 8fa69b4

Please sign in to comment.