Skip to content

Commit 8fa69b4

Browse files
npm1TallTed
andauthored
Apply suggestions from code review
Co-authored-by: Ted Thibodeau Jr <[email protected]>
1 parent db413f2 commit 8fa69b4

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

spec/index.bs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -472,7 +472,7 @@ To <dfn>compute the connection status</dfn> given an {{IdentityProviderConfig}}
472472
1. Let |triple| be (|rpOrigin|, |idpOrigin|, |accountId|).
473473
1. If [=connected accounts set=] [=list/contains=] |triple| , return
474474
[=compute the connection status/connected=].
475-
1. Otherwise, if account| [=map/contains=] {{IdentityProviderAccount/approved_clients}},
475+
1. Otherwise, if |account| [=map/contains=] {{IdentityProviderAccount/approved_clients}},
476476
|account|'s {{IdentityProviderAccount/approved_clients}} [=list/contains=] |provider|'s
477477
{{IdentityProviderConfig/clientId}}, and third party cookies are available for |idpOrigin|
478478
on |rpOrigin|, return [=compute the connection status/connected=].
@@ -1421,9 +1421,9 @@ When invoking the {{IdentityProvider/getUserInfo()}} method given an {{IdentityP
14211421
* There does not exist an account |account| such that [=compute the connection status=] of
14221422
|provider|, |account|, and |globalObject| returns
14231423
[=compute the connection status/connected=]. This check can be performed by iterating
1424-
over the [=connected accounts set=] or by keeping a separate data structure to make this
1425-
lookup fast.
1426-
* There is third party cookie access for |configUrl|'s [=url/origin=] on |document|'s
1424+
over the [=connected accounts set=] or by keeping a separate data structure for faster
1425+
lookup results.
1426+
* There is no third party cookie access for |configUrl|'s [=url/origin=] on |document|'s
14271427
[=Document/origin=].
14281428
1. If |configUrl| is failure, throw an "{{InvalidStateError}}" {{DOMException}}.
14291429
1. If |document|'s [=Document/origin=] is not [=same origin=] as |configUrl|'s [=url/origin=],

0 commit comments

Comments
 (0)