Bug 62806 – Set referrer sooner on a ResourceRequest from a worker

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Nate Chapin]

Currently, we have an optionalOutgoingReferrer parameter on several functions to allow a WorkerThreadableLoader to override the referrer of its requests.  

I think we should be able to call setHTTPReferrer() on the ResourceRequest as soon as we are in the main thread and pass the overriding referrer that way, meaning one less optional parameter in a few places.

Patch coming shortly.

[Nate Chapin]

Created attachment 97471 [details]
patch

[Alexey Proskuryakov]

Comment on attachment 97471 [details]
patch

I can't follow how Referer setting logic works with this patch. For example, will we be getting https workers' URLs sent to the server in this field now? Or was it a problem even before this patch?

[Nate Chapin]

(In reply to comment #2)
> (From update of attachment 97471 [details])
> I can't follow how Referer setting logic works with this patch. For example, will we be getting https workers' URLs sent to the server in this field now? Or was it a problem even before this patch?

The diff cut off at a lousy place.  The next line in SubresourceLoader.cpp should enforce referrer blocking regardless of whether we're using the document's outgoingReferrer of the worker's override: http://trac.webkit.org/browser/trunk/Source/WebCore/loader/SubresourceLoader.cpp#L94

[Alexey Proskuryakov]

Looks good to me. David, do you remember if there was a reason why we didn't implement it this way originally?

[David Levin]

(In reply to comment #4)
> Looks good to me. David, do you remember if there was a reason why we didn't implement it this way originally?

In short, it was a bad oversight on my part.

As best I can remember, my mental rough outline was to add it into ResourceRequest but was discouraged from adding more there.  Unfortunately (for no good reason), I must not have checked to see that it was already there. 

This is much better.

[WebKit Review Bot]

Comment on attachment 97471 [details]
patch

Clearing flags on attachment: 97471

Committed r89086: <http://trac.webkit.org/changeset/89086>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.