|
OnDownloadStarted takes DownloadItem* instead of DownloadId
PS1@r160378
Reviewers:
sky: content/public/test, content/browser/renderer_host/resource_dispatcher_host_impl.cc
binji: chrome/browser/ui/intents
bauerb: chrome/browser/plugins
finnur: chrome/browser/extensions
Done:
rdsmith: *
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=160971
Total comments: 8
Total comments: 6
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+71 lines, -79 lines) |
Patch |
|
M |
chrome/browser/download/download_browsertest.cc
|
View
|
|
1 chunk |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/extensions/api/downloads/downloads_api.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
chrome/browser/extensions/api/downloads/downloads_api.cc
|
View
|
|
1 chunk |
+7 lines, -4 lines |
0 comments
|
Download
|
|
M |
chrome/browser/extensions/extension_browsertest.cc
|
View
|
1
2
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
|
M |
chrome/browser/extensions/webstore_installer.h
|
View
|
|
2 chunks |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/extensions/webstore_installer.cc
|
View
|
|
2 chunks |
+11 lines, -19 lines |
0 comments
|
Download
|
|
M |
chrome/browser/plugins/plugin_installer.h
|
View
|
1
|
2 chunks |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/plugins/plugin_installer.cc
|
View
|
1
|
2 chunks |
+7 lines, -12 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/intents/web_intent_picker_controller.h
|
View
|
1
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
|
M |
content/browser/download/download_manager_impl.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
content/browser/download/download_manager_impl.cc
|
View
|
1
2
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
content/browser/download/download_resource_handler.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
content/browser/download/download_resource_handler.cc
|
View
|
1
2
|
5 chunks |
+11 lines, -10 lines |
0 comments
|
Download
|
|
M |
content/browser/renderer_host/resource_dispatcher_host_impl.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
content/public/browser/download_manager.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
content/public/browser/download_url_parameters.h
|
View
|
1
2
|
3 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
|
M |
content/public/browser/resource_dispatcher_host.h
|
View
|
|
3 chunks |
+8 lines, -6 lines |
0 comments
|
Download
|
|
M |
content/public/test/download_test_observer.h
|
View
|
|
2 chunks |
+3 lines, -5 lines |
0 comments
|
Download
|
|
M |
content/public/test/download_test_observer.cc
|
View
|
1
2
3
|
2 chunks |
+5 lines, -3 lines |
0 comments
|
Download
|
|
M |
content/public/test/mock_download_manager.h
|
View
|
|
1 chunk |
+2 lines, -3 lines |
0 comments
|
Download
|
|
M |
content/public/test/mock_download_manager.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
Total messages: 13 (0 generated)
|