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

Unified Diff: content/common/url_loader.mojom

Issue 2503813002: Fix and refactor downloaded file handling in the loading stack (Closed)
Patch Set: move to dtor Created 4 years, 1 month 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/common/url_loader.mojom
diff --git a/content/common/url_loader.mojom b/content/common/url_loader.mojom
index c8d8ac3a9c50f8ce4c599ae1829dbe7aebb20a65..f77988b238bd2d74a1d6a29e8b682de11ce9bec8 100644
--- a/content/common/url_loader.mojom
+++ b/content/common/url_loader.mojom
@@ -24,9 +24,18 @@ interface URLLoader {
FollowRedirect();
};
+// Opaque handle passed from the browser process to a child process to manage
+// the lifetime of temporary files used for download_to_file resource loading.
+// When the message pipe for this interface is closed, the browser process will
+// clean up the corresponding temporary file.
+interface DownloadedTempFile {
+};
+
+
interface URLLoaderClient {
- // Called when the response head is received.
- OnReceiveResponse(URLResponseHead head);
+ // Called when the response head is received. |downloaded_file| is non-null in
+ // the 'download_to_file' case.
+ OnReceiveResponse(URLResponseHead head, DownloadedTempFile? downloaded_file);
// Called when the request has been redirected. The receiver is expected to
// call FollowRedirect or cancel the request.
« no previous file with comments | « content/child/resource_dispatcher.cc ('k') | content/renderer/service_worker/service_worker_context_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698