|
[Downloads] Persist ETag and Last-Modified headers.
These response headers are used for partial download resumption. If an
interrupted download had a valid strong ETag header and the server
supports byte range requests, then a subsequent downloads resumption
attempt could fetch part of a resource instead of the entire resource.
While the Last-Modified header is also used for validation currently,
its use will be discontinued since a matching Last-Modifed value does
not imply byte-wise equivalence.
Without persisting these headers, partial downloads resumption will
only work for downloads that are interrupted and resumed during the
same browser session.
BUG= 7648
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216493
Total comments: 3
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+426 lines, -235 lines) |
Patch |
|
M |
chrome/browser/download/download_browsertest.cc
|
View
|
1
2
3
|
3 chunks |
+13 lines, -4 lines |
0 comments
|
Download
|
|
M |
chrome/browser/download/download_history.cc
|
View
|
|
3 chunks |
+6 lines, -0 lines |
0 comments
|
Download
|
|
M |
chrome/browser/download/download_history_unittest.cc
|
View
|
1
|
15 chunks |
+206 lines, -183 lines |
0 comments
|
Download
|
|
M |
chrome/browser/extensions/api/downloads/downloads_api_browsertest.cc
|
View
|
1
2
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
|
M |
chrome/browser/history/download_database.h
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
chrome/browser/history/download_database.cc
|
View
|
1
|
9 chunks |
+52 lines, -36 lines |
0 comments
|
Download
|
|
M |
chrome/browser/history/download_row.h
|
View
|
|
2 chunks |
+8 lines, -0 lines |
0 comments
|
Download
|
|
M |
chrome/browser/history/download_row.cc
|
View
|
|
2 chunks |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
chrome/browser/history/history_database.cc
|
View
|
|
2 chunks |
+13 lines, -4 lines |
0 comments
|
Download
|
|
M |
chrome/browser/history/history_unittest.cc
|
View
|
1
2
3
|
3 chunks |
+69 lines, -0 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/webui/downloads_dom_handler_browsertest.cc
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
A + |
chrome/test/data/History/history.27.sql
|
View
|
|
1 chunk |
+4 lines, -5 lines |
0 comments
|
Download
|
|
M |
chrome/test/data/download-test1.lib.mock-http-headers
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/browser/download/download_item_factory.h
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/browser/download/download_item_impl.h
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/browser/download/download_item_impl.cc
|
View
|
|
2 chunks |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/browser/download/download_manager_impl.h
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/browser/download/download_manager_impl.cc
|
View
|
|
4 chunks |
+8 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/browser/download/download_manager_impl_unittest.cc
|
View
|
|
3 chunks |
+6 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/public/browser/download_manager.h
|
View
|
|
2 chunks |
+2 lines, -1 line |
0 comments
|
Download
|
|
M |
content/public/test/mock_download_manager.h
|
View
|
|
3 chunks |
+6 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/public/test/mock_download_manager.cc
|
View
|
|
5 chunks |
+10 lines, -2 lines |
0 comments
|
Download
|
Total messages: 12 (0 generated)
|