[webkit-reviews] review denied: [Bug 86000] Take referrer policy into account when clearing the referrer header : [Attachment 141552] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat May 12 00:38:53 PDT 2012


Alexey Proskuryakov <ap at webkit.org> has denied	review:
Bug 86000: Take referrer policy into account when clearing the referrer header
https://bugs.webkit.org/show_bug.cgi?id=86000

Attachment 141552: Patch
https://bugs.webkit.org/attachment.cgi?id=141552&action=review

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
I suggested this, but now I realize that this is a fragile solution. Every
ResourceHandleClient now needs to implement this, and ResourceHandle may still
need this when its client is null.

Specifically, BlobResourceSynchronousLoader is a ResourceHandleClient, and need
to implement the same logic. And so is ApplicationCacheGroup.

I guess that we need to add a new client call, so that clients that care would
be able to customize the behavior that's still in ResourceHandle. Sorry for a
false lead.


More information about the webkit-reviews mailing list