[webkit-reviews] review granted: [Bug 55903] SubresourceLoader should expose a way to set the outgoing referer/origin. : [Attachment 84988] Patch

[bugzilla-daemon at webkit.org]

Adam Barth <abarth at webkit.org> has granted David Levin <levin at chromium.org>'s
request for review:
Bug 55903: SubresourceLoader should expose a way to set the outgoing
referer/origin.
https://bugs.webkit.org/show_bug.cgi?id=55903

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

------- Additional Comments from Adam Barth <abarth at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=84988&action=review

> Source/WebCore/loader/SubresourceLoader.cpp:63
> -PassRefPtr<SubresourceLoader> SubresourceLoader::create(Frame* frame,
SubresourceLoaderClient* client, const ResourceRequest& request,
SecurityCheckPolicy securityCheck, bool sendResourceLoadCallbacks, bool
shouldContentSniff)
> +PassRefPtr<SubresourceLoader> SubresourceLoader::create(Frame* frame,
SubresourceLoaderClient* client, const ResourceRequest& request,
SecurityCheckPolicy securityCheck, bool sendResourceLoadCallbacks, bool
shouldContentSniff, String outgoingReferrer)

We usually pass String by const reference.