Bug 248566 - Rename request initiator and initiatorName to initiatorType
Summary: Rename request initiator and initiatorName to initiatorType
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Matthew Finkel
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-11-30 15:43 PST by Matthew Finkel
Modified: 2022-12-05 09:36 PST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Finkel 2022-11-30 15:43:21 PST
Currently we use both |initiator| and |initiatorName| for representing roughly the same idea of answering the question: "what triggered a resource request". These values are used internally and for supporting ResourceTiming. However, in the specs this idea is named |initiatorType| [0]. In an effort to be consistent internally, and reduce confusion with respect to the spec, I'd like to rename the current usage.

This is also the first step in adding support for Fetch's |initiator| [1], which is another vague classification of a request, but it is different from |initiatorType|.

[0] https://fetch.spec.whatwg.org/#request-initiator-type
[1] https://fetch.spec.whatwg.org/#concept-request-initiator
Comment 1 Radar WebKit Bug Importer 2022-11-30 15:43:34 PST
<rdar://problem/102832673>
Comment 2 Matthew Finkel 2022-11-30 20:43:59 PST
Pull request: https://github.com/webkit/WebKit/pull/7007
Comment 3 EWS 2022-12-05 09:35:40 PST
Committed 257374@main (7d03a4356c05): <https://commits.webkit.org/257374@main>

Reviewed commits have been landed. Closing PR #7007 and removing active labels.