Issue 1958383002: [Merge M51][Downloads] Set the Referrer header for resumed requests.

Issue 1958383002: [Merge M51][Downloads] Set the Referrer header for resumed requests. (Closed)

Created:
4 years, 7 months ago by asanka

Modified:
4 years, 7 months ago

Reviewers:
svaldez, jam

CC:
chromium-reviews

Base URL:
https://chromium.googlesource.com/chromium/src.git@2704

Target Ref:
refs/pending/branch-heads/2704

Project:
chromium

Visibility:
Public.

[Merge M51][Downloads] Set the Referrer header for resumed requests. During downloads resumption, the newly created URLRequest was not getting the Referrer header set. Now it is. R=jam@chromium.org,svaldez@chromium.org BUG=603670 Review URL: https://codereview.chromium.org/1891623003 Cr-Commit-Position: refs/heads/master@{#389636} (cherry picked from commit f7ae1f45615bc167f2081079557322eefba02c0b) Committed: https://chromium.googlesource.com/chromium/src/+/69bc08ef5cd48356e6a254ff8366159cdeeefc1a

Created: 4 years, 7 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+177 lines, -38 lines)			Patch
M	content/browser/download/download_browsertest.cc	View	11 chunks	+124 lines, -24 lines	0 comments	Download
M	content/browser/download/download_item_impl.cc	View	3 chunks	+11 lines, -4 lines	0 comments	Download
M	content/public/test/test_download_request_handler.h	View	1 chunk	+18 lines, -1 line	0 comments	Download
M	content/public/test/test_download_request_handler.cc	View	5 chunks	+23 lines, -8 lines	0 comments	Download
A +	content/test/data/download/download-link.html	View	1 chunk	+2 lines, -2 lines	0 comments	Download
A +	content/test/data/download/download-link.html.mock-http-headers	View	0 chunks	+-1 lines, --1 lines	0 comments	Download

Total messages: 2 (1 generated)

asanka

4 years, 7 months ago (2016-05-10 03:46:05 UTC) #2

Message was sent while issue was closed.

Committed patchset #1 (id:1) manually as
69bc08ef5cd48356e6a254ff8366159cdeeefc1a.