New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add consideration of browser permissions framework for extension processing #763
Conversation
index.bs
Outdated
## Browser Permissions Framework and Extensions ## (#browser-permissions-framework-extensions) | ||
|
||
The Web Authentication API should leverage the browser permissions framework as much as possible when obtaining user permissions for | ||
certain extensions. An example is the location extrension, which can leverrage the existing browser permissions framework for the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typos: "extrension" -> "extension", "leverrage" -> "leverage"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
@gmandyam please resolve the merge conflicts. Then I will merge the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the unrelated and unintended change from the PR. After that, I'm OK with it being merged, as at that point it will only be adding Security Considerations.
latitude: float64, | ||
longitude: float64, | ||
altitude: float64 | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An unrelated change appears to have been made in this PR. Please remove this extraneous change, as it will cause merge conflicts with other PRs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR #771 replaces this |
In response to #133
Preview | Diff