[webkit-reviews] review denied: [Bug 46529] Output X-Purpose header for prefetch requests : [Attachment 68781] and move the timer into onload, making this test work better outside of harness

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has denied  review:
Bug 46529: Output X-Purpose header for prefetch requests
https://bugs.webkit.org/show_bug.cgi?id=46529

Attachment 68781: and move the timer into onload, making this test work better
outside of harness
https://bugs.webkit.org/attachment.cgi?id=68781&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
Per contention above, r-.

Why would we want to let servers know it's a prefetch request?	We don't let
them know other things about how we're going to display or not-display their
content.

For example, we don't tell servers that we're about to display their content in
a display:none iframe.	Or on what screen size, etc.  Proxies also don't
necessarily tell servers that they're a proxy and thus not displaying the
content...  This seems like an invalid bug to me.

A request is a request.  If servers feel they need to filter out requests based
on what was done with that request, it seems they should be gathering that
information differently (like via JavaScript?).