-
Notifications
You must be signed in to change notification settings - Fork 381
Add a render-blocking boolean flag to the request concept #1432
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
2f45e67 to
74c8982
Compare
|
@annevk @yoavweiss PTAL |
annevk
left a comment
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.
Thanks, this looks good to me modulo some nits.
xiaochengh
left a comment
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.
Thanks for reviewing!
annevk
left a comment
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.
Thanks, looks great!
This patch follows from the fetch spec revision: whatwg/fetch#1432 Bug: 1271296 Change-Id: I3f56e921e238a7e28774d5407a78d4df4f4145d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3627294 Reviewed-by: Yoav Weiss <yoavweiss@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by: Patrick Meenan <pmeenan@chromium.org> Cr-Commit-Position: refs/heads/main@{#1007082}
See whatwg/fetch#1433 and whatwg/fetch#1432 for background.
See whatwg/fetch#1433 and whatwg/fetch#1432 for background.
Fixes #1433. The HTML-side counterpart is whatwg/html#7885.
This is also a minor follow-up of whatwg/html#7474.
blocking=renderattribute html#7474blocking=renderattribute html#7474(See WHATWG Working Mode: Changes for more details.)
Preview | Diff