Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(890)

Unified Diff: content/browser/loader/mojo_async_resource_handler.cc

Issue 2561743003: Use associated interface on DownloadedTempFile (Closed)
Patch Set: typo fix Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/loader/mojo_async_resource_handler.cc
diff --git a/content/browser/loader/mojo_async_resource_handler.cc b/content/browser/loader/mojo_async_resource_handler.cc
index 0d103654468e3923424646aa51031246b07daeef..8f65ed3bd566d77cb134a0d99cf25d20c0f4ddb0 100644
--- a/content/browser/loader/mojo_async_resource_handler.cc
+++ b/content/browser/loader/mojo_async_resource_handler.cc
@@ -171,10 +171,10 @@ bool MojoAsyncResourceHandler::OnResponseStarted(ResourceResponse* response,
response->head.response_start = base::TimeTicks::Now();
sent_received_response_message_ = true;
- mojom::DownloadedTempFilePtr downloaded_file_ptr;
+ mojom::DownloadedTempFileAssociatedPtrInfo downloaded_file_ptr;
if (!response->head.download_file_path.empty()) {
mmenke 2017/01/18 20:08:53 Hrm....I assume we never do this for main frames (
tzik 2017/01/19 13:04:41 Do you mean TransferredNavigation case? Right, it'
mmenke 2017/01/19 13:09:08 I mean cases in which we'd want to transfer the UR
- downloaded_file_ptr = DownloadedTempFileImpl::Create(info->GetChildID(),
- info->GetRequestID());
+ downloaded_file_ptr = DownloadedTempFileImpl::Create(
+ binding_.associated_group(), info->GetChildID(), info->GetRequestID());
rdh_->RegisterDownloadedTempFile(info->GetChildID(), info->GetRequestID(),
response->head.download_file_path);
}
« no previous file with comments | « content/browser/loader/downloaded_temp_file_impl.cc ('k') | content/browser/loader/resource_dispatcher_host_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698