Fix referrer policy passing during redirects

Fix referrer policy passing during redirects

We a redirect commits, we should take the referrer policy from the
request we actually send and not from the one we intended to send.

Also, when the renderer tells the browser about a new navigation, always
update the referrer together with the URL

Depends on https://codereview.chromium.org/126753002/ in blink

BUG=331941
R=nasko@chromium.org
TEST=browser_tests:ReferrerPolicyTest.*

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=244138

[jochen (gone - plz use gerrit), 2014-01-08 14:57:42]

plz review

I'll fire of tryjobs once the blink side landed and rolled in.

[nasko, 2014-01-08 17:11:08]

LGTM with one note.

https://codereview.chromium.org/128173002/diff/1/content/renderer/render_view...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/128173002/diff/1/content/renderer/render_view...
content/renderer/render_view_impl.cc:1965: // would be nice if we could get the
real referrer from somewhere.
Is this comment still valid after your change?

[jochen (gone - plz use gerrit), 2014-01-08 19:06:04]

https://codereview.chromium.org/128173002/diff/1/content/renderer/render_view...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/128173002/diff/1/content/renderer/render_view...
content/renderer/render_view_impl.cc:1965: // would be nice if we could get the
real referrer from somewhere.
On 2014/01/08 17:11:09, nasko wrote:
> Is this comment still valid after your change?

Yes, if the referrer was stripped, it's not on any request.

However, I disagree that it would be nice to have the real referrer. It would
just be a mess to make sure we don't accidentially leak it. I'll remove the
comment (and closed the bug)

[commit-bot: I haz the power, 2014-01-10 05:15:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jochen@chromium.org/128173002/80001

[commit-bot: I haz the power, 2014-01-10 07:09:39]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2014-01-10 09:02:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jochen@chromium.org/128173002/360001

[commit-bot: I haz the power, 2014-01-10 11:02:30]

Change committed as 244138