Skip to content
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

Resource Timing: Expose firstInterimResponseStart #37984

Merged
merged 1 commit into from
Jan 19, 2023

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 16, 2023

This adds an entry to PerformanceResourceTiming:

  • firstInterimResponseStart: the time of the first early-hints header

It also changes the meaning of responseStart to be the first
non-informational header (non-103).

Implemented for Quic, Spdy and HTTP.

All behind a feature runtime flag (ResourceTimingInterimResponseTimes)

Spec issue: w3c/resource-timing#345

Bug: 1402089
Change-Id: I2f050788515959e3576f3cf2bd8df13ff848090a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4165825
Reviewed-by: Bence Béky <[email protected]>
Commit-Queue: Noam Rosenthal <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1094571}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot changed the title Expose first interim response timing Resource Timing: Expose firstInterimResponseStart Jan 16, 2023
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-4165825 branch 5 times, most recently from c65e55d to 1ddc019 Compare January 19, 2023 16:46
This adds an entry to PerformanceResourceTiming:
- firstInterimResponseStart: the time of the first early-hints header

It also changes the meaning of responseStart to be the first
non-informational header (non-103).

Implemented for Quic, Spdy and HTTP.

All behind a feature runtime flag (ResourceTimingInterimResponseTimes)

Spec issue: w3c/resource-timing#345

Bug: 1402089
Change-Id: I2f050788515959e3576f3cf2bd8df13ff848090a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4165825
Reviewed-by: Bence Béky <[email protected]>
Commit-Queue: Noam Rosenthal <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1094571}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants