Use associated interface on DownloadedTempFile

content/browser/loader/mojo_async_resource_handler.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/loader/mojo_async_resource_handler.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/containers/hash_tables.h"
@@ skipping 153 matching lines @@
   }
 
   NetLogObserver::PopulateResponseInfo(request(), response);
   response->head.encoded_data_length = request()->raw_header_size();
   reported_total_received_bytes_ = response->head.encoded_data_length;
 
   response->head.request_start = request()->creation_time();
   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 (Or sub frames)?  We'll want to
be passing main frame URLLoaders from the renderer to the browser in response to
OnResponseStarted calls, which we can't do once we have associated interfaces, I
believe?

[tzik, 2017/01/19 13:04:41]

Do you mean TransferredNavigation case? Right, it's only for the navigation and
a request for the main resource, for which we don't use download-to-file mode.

[mmenke, 2017/01/19 13:09:08]

I mean cases in which we'd want to transfer the URLLoader around.  For
navigations (Both main frame, with PlzNavigate, and subframes, with
PlzNavigate+OOPIF), we'd want to drive the URLLoader from the navigation code,
and then pass it to the renderer (Or download subsystem, if we discover it's a
download)

-    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);
   }
 
   url_loader_client_->OnReceiveResponse(response->head,
                                         std::move(downloaded_file_ptr));
   return true;
 }
 
 bool MojoAsyncResourceHandler::OnWillStart(const GURL& url, bool* defer) {
@@ skipping 283 matching lines @@
     mojom::URLLoaderAssociatedRequest mojo_request,
     mojom::URLLoaderClientAssociatedPtr url_loader_client) {
   binding_.Unbind();
   binding_.Bind(std::move(mojo_request));
   binding_.set_connection_error_handler(
       base::Bind(&MojoAsyncResourceHandler::Cancel, base::Unretained(this)));
   url_loader_client_ = std::move(url_loader_client);
 }
 
 }  // namespace content