Bug 102064
| Summary: | Meta: add initiator to CachedResourceRequest | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Marja Hölttä <marja> |
| Component: | New Bugs | Assignee: | Marja Hölttä <marja> |
| Status: | REOPENED | ||
| Severity: | Normal | CC: | abarth, battre, jochen, simonjam |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | 101935 | ||
| Bug Blocks: | |||
Marja Hölttä
The initiator information is needed for bug 92761 and for bug 84883.
The current plan is to split this in multiple patches:
- Add initiator to CachedResourceRequest for simple cases (not ThreadableLoaders) (bug 101935).
- Make ThreadableLoaders also set the initiator, assert that the initiator is always set.
- Add tests (make CachedResourceLoader record the requests and their initiators, read the recorded data to check that the initiators are set).
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
James Simonsen
FYI, I have plenty of initiator tests for Resource Timing. I will land them when Resource Timing goes in.
Adam Barth
> FYI, I have plenty of initiator tests for Resource Timing. I will land them when Resource Timing goes in.
I'd rather not block having tests on implementing Resource Timing because I don't understand how you plan to implement Resource Timing.
Marja Hölttä
Then I'll close this, and I'll add tests as part of reviving bug 92761.
Marja Hölttä
What, no, not this bug.