search

w3c / resource-timing

Resource Timing
https://w3c.github.io/resource-timing/
Other
121 stars 35 forks source link

Minor clarity improvement in Privacy & Security section #307

Closed patrickhulce closed 2 years ago

patrickhulce commented 2 years ago

Just a minor typo fix :) this was fixed before this could be merged, so now just clarity improvement

Aside, this sentence was a bit hard for me to parse. Is something like...

Though the PerformanceResourceTiming interface gives timing information for resources in a document, the load event on resources can already measure timing to determine cache hits and misses in a limited fashion, and the cross-origin restrictions in [=/HTTP Fetch=] prevent the leakage of any additional information.

  1. Accurate?
  2. A possible alternative that could be considered?

Preview | Diff

npm1 commented 2 years ago

I think your suggestion is clearer and still accurate, do you mind updating your PR to do that? Also, could you link your GitHub account to W3C so that the iPR checks pass? I could mark this as non substantive but might be worth doing in case you plan to send other changes later :)

npm1 commented 2 years ago

Did you get a chance to link your GitHub to W3C? Otherwise this can be marked non-substantive for merging.

patrickhulce commented 2 years ago

Thanks!

Did you get a chance to link your GitHub to W3C? Otherwise this can be marked non-substantive for merging.

Go ahead and mark and as non-substantive. I think my Invited Expert status did not renew since it's been roughly year since heavy involvement, so I'm locked out of my account atm.

w3cbot commented 2 years ago

yoavweiss marked as non substantive for IPR from ash-nazg.

patrickhulce commented 2 years ago

Thanks @yoavweiss and @npm1 :)