Send the correct referrer and policy when saving a file via PPAPI

Send the correct referrer and policy when saving a file via PPAPI

BUG=124750
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137249

[jochen (gone - plz use gerrit), 2012-05-12 00:15:34]

Randy, Gene, please review

Avi: owner stuff

[Randy Smith (Not in Mondays), 2012-05-12 00:21:23]

Swapping in Ben for me as it's save package related.

[jochen (gone - plz use gerrit), 2012-05-12 00:21:58]

btw, any suggestion how to test this would be welcome :)

[Avi (use Gerrit), 2012-05-12 06:17:10]

owner rubberstamp lgtm; get approval from someone who's qualified.

[benjhayden, 2012-05-14 13:56:20]

Looks like this CL is currently just a small bit of plumbing, doesn't do what
the description says it does.

http://codereview.chromium.org/10377118/diff/1/content/browser/web_contents/w...
File content/browser/web_contents/web_contents_impl.cc (right):

http://codereview.chromium.org/10377118/diff/1/content/browser/web_contents/w...
content/browser/web_contents/web_contents_impl.cc:2722: // TODO(jochen): Pass in
the referrer policy as well.
Any reason not to do this in this cl?

[jochen (gone - plz use gerrit), 2012-05-14 14:11:22]

http://codereview.chromium.org/10377118/diff/1/content/browser/web_contents/w...
File content/browser/web_contents/web_contents_impl.cc (right):

http://codereview.chromium.org/10377118/diff/1/content/browser/web_contents/w...
content/browser/web_contents/web_contents_impl.cc:2722: // TODO(jochen): Pass in
the referrer policy as well.
On 2012/05/14 13:56:20, benjhayden_chromium wrote:
> Any reason not to do this in this cl?

Yes, because then I need to change all places that use DownloadUrlParameters at
the same time. The idea is to first get the referrer to all places where
DownloadUrlParameters is created in individual CLs, and once that's done, switch
DownloadUrlParameters to actually use that

[benjhayden, 2012-05-14 14:30:17]

LGTM

http://codereview.chromium.org/10232010/
You don't need to update all of DUP's clients, just the ones that set referrer.
I don't think that that would be too big for one CL, but it's up to you.

[gene, 2012-05-15 19:34:03]

lgtm

[commit-bot: I haz the power, 2012-05-15 19:35:00]

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

[commit-bot: I haz the power, 2012-05-15 21:23:48]

Change committed as 137249