-
Notifications
You must be signed in to change notification settings - Fork 2.4k
WebNN API - October 2025 update - 2 #28157
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
Conversation
Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large 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.
👍
@Elchi3 to double-check this now matches what OWD collector reports with the latest build.
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.
LGTM but the reordering changes in api/Navigator.json
seem weird to me. Tests pass for me without these changes. Could you revert that?
Thank you @Elchi3, the unexpected reordering changes in |
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.
Wonderful, thank you @ibelem 👍
This is a follow-up PR of #28133, add missing items for the WebNN API to ensure comprehensive coverage:
Summary
Also updated to
"chrome_android": "mirror"
from"chrome_android": { "version_added": false }
.There are some order changes in
api/Navigator.json
, caused by running"fix": "npm run format:fix && npm run lint:fix"
@anssiko @Elchi3 PTAL, thanks!