Bug 229167 - Report resource timing when SubresourceLoader::checkForHTTPStatusCodeError returns true
Summary: Report resource timing when SubresourceLoader::checkForHTTPStatusCodeError re...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alex Christensen
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-08-16 15:25 PDT by Alex Christensen
Modified: 2021-08-16 19:44 PDT (History)
4 users (show)

See Also:


Attachments
Patch (8.67 KB, patch)
2021-08-16 15:28 PDT, Alex Christensen
no flags Details | Formatted Diff | Diff
Patch (9.73 KB, patch)
2021-08-16 15:49 PDT, Alex Christensen
no flags Details | Formatted Diff | Diff
Patch (9.94 KB, patch)
2021-08-16 15:52 PDT, Alex Christensen
no flags Details | Formatted Diff | Diff
Patch (11.28 KB, patch)
2021-08-16 17:14 PDT, Alex Christensen
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Christensen 2021-08-16 15:25:45 PDT
Report resource timing when SubresourceLoader::checkForHTTPStatusCodeError returns true
Comment 1 Alex Christensen 2021-08-16 15:28:31 PDT
Created attachment 435633 [details]
Patch
Comment 2 Alex Christensen 2021-08-16 15:49:15 PDT
Created attachment 435637 [details]
Patch
Comment 3 Alex Christensen 2021-08-16 15:52:07 PDT
Created attachment 435638 [details]
Patch
Comment 4 Chris Dumez 2021-08-16 16:28:20 PDT
Comment on attachment 435638 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=435638&action=review

r=me

> Source/WebCore/ChangeLog:10
> +        those resources' timing data.  This wasn't a problem before r281110 because we would've discareded it anyways, but

typo: discareded
Comment 5 Alex Christensen 2021-08-16 17:14:22 PDT
Created attachment 435647 [details]
Patch
Comment 6 EWS 2021-08-16 19:43:15 PDT
Committed r281125 (240579@main): <https://commits.webkit.org/240579@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 435647 [details].
Comment 7 Radar WebKit Bug Importer 2021-08-16 19:44:22 PDT
<rdar://problem/82009989>