Bug 257407 - Web Inspector: XHR request with same URL as main resource should have type XHR
Summary: Web Inspector: XHR request with same URL as main resource should have type XHR
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Yury Semikhatsky
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-05-26 18:08 PDT by Yury Semikhatsky
Modified: 2023-06-01 18:56 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yury Semikhatsky 2023-05-26 18:08:48 PDT
XHRs requests with the same url as main resource should have type XHR rather than Document.
This was fixed in https://bugs.webkit.org/show_bug.cgi?id=68646 but regressed at some point later.
Original report in Playwright: https://github.com/microsoft/playwright/issues/22812
Comment 1 Yury Semikhatsky 2023-05-26 18:18:52 PDT
Pull request: https://github.com/WebKit/WebKit/pull/14417
Comment 2 EWS 2023-05-30 11:37:58 PDT
Committed 264686@main (883556d2265f): <https://commits.webkit.org/264686@main>

Reviewed commits have been landed. Closing PR #14417 and removing active labels.
Comment 3 Radar WebKit Bug Importer 2023-05-30 11:38:18 PDT
<rdar://problem/110016863>
Comment 4 Fujii Hironori 2023-06-01 13:59:14 PDT
http/tests/inspector/network/xhr-request-type.html is failing. Could you take a look? 
https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Finspector%2Fnetwork%2Fxhr-request-type.html
Comment 5 Yury Semikhatsky 2023-06-01 17:17:37 PDT
Hmm, the .json resource is returned in base64 on some platforms, I'll take a look.
Comment 6 Yury Semikhatsky 2023-06-01 18:09:53 PDT
The test is passing locally on M1 and macOS 13. I wonder if it has to do with the web server config.
Comment 7 Fujii Hironori 2023-06-01 18:56:31 PDT
This test failure is reproducible with WinCairo.
The web server doesn't know .json filename suffix.
Renaming data.json to data.js can works around the server configuration problem.