Confirmed users
112
edits
DOM/WebIDL Review Checklist: Difference between revisions
→Spec makes sense: add secure context requirement
(Add link to intent to implement) |
(→Spec makes sense: add secure context requirement) |
||
| Line 15: | Line 15: | ||
* Whether arguments and dictionary members are optional or not needs to make sense. | * Whether arguments and dictionary members are optional or not needs to make sense. | ||
* <code>Exposed</code> values should make sense. | * <code>Exposed</code> values should make sense. | ||
* It uses <code>SecureContext</code> (see [https://blog.mozilla.org/security/2018/01/15/secure-contexts-everywhere/ Secure Contexts Everywhere] if not). | |||
* The order in which checks that lead to exceptions being thrown happen should be defined in cases when the method can have multiple exceptional conditions. | * The order in which checks that lead to exceptions being thrown happen should be defined in cases when the method can have multiple exceptional conditions. | ||
* The spec should almost certainly not be using the term "active document". If it is, think extra carefully about what happens when that document is not same-origin with any of the other things the spec is talking about. | * The spec should almost certainly not be using the term "active document". If it is, think extra carefully about what happens when that document is not same-origin with any of the other things the spec is talking about. | ||