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

Side by Side Diff: content/common/url_loader.mojom

Issue 2658583002: Revert of Use associated interface on DownloadedTempFile (Closed)
Patch Set: Rebased. Created 3 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 module content.mojom; 5 module content.mojom;
6 6
7 [Native] 7 [Native]
8 struct URLRequest; 8 struct URLRequest;
9 9
10 [Native] 10 [Native]
(...skipping 17 matching lines...) Expand all
28 // the lifetime of temporary files used for download_to_file resource loading. 28 // the lifetime of temporary files used for download_to_file resource loading.
29 // When the message pipe for this interface is closed, the browser process will 29 // When the message pipe for this interface is closed, the browser process will
30 // clean up the corresponding temporary file. 30 // clean up the corresponding temporary file.
31 interface DownloadedTempFile { 31 interface DownloadedTempFile {
32 }; 32 };
33 33
34 34
35 interface URLLoaderClient { 35 interface URLLoaderClient {
36 // Called when the response head is received. |downloaded_file| is non-null in 36 // Called when the response head is received. |downloaded_file| is non-null in
37 // the 'download_to_file' case. 37 // the 'download_to_file' case.
38 OnReceiveResponse(URLResponseHead head, 38 OnReceiveResponse(URLResponseHead head, DownloadedTempFile? downloaded_file);
39 associated DownloadedTempFile? downloaded_file);
40 39
41 // Called when the request has been redirected. The receiver is expected to 40 // Called when the request has been redirected. The receiver is expected to
42 // call FollowRedirect or cancel the request. 41 // call FollowRedirect or cancel the request.
43 OnReceiveRedirect(URLRequestRedirectInfo redirect_info, URLResponseHead head); 42 OnReceiveRedirect(URLRequestRedirectInfo redirect_info, URLResponseHead head);
44 43
45 // Called when some data from a resource request has been downloaded to the 44 // Called when some data from a resource request has been downloaded to the
46 // file. This is only called in the 'download_to_file' case and replaces 45 // file. This is only called in the 'download_to_file' case and replaces
47 // OnStartLoadingResponseBody in the call sequence in that case. 46 // OnStartLoadingResponseBody in the call sequence in that case.
48 // TODO(yhirano): Remove |encoded_length| and use OnTransferSizeUpdated 47 // TODO(yhirano): Remove |encoded_length| and use OnTransferSizeUpdated
49 // instead. 48 // instead.
(...skipping 21 matching lines...) Expand all
71 70
72 // Called when the loader starts loading response body. This is called after 71 // Called when the loader starts loading response body. This is called after
73 // OnReceiveResponse is called. 72 // OnReceiveResponse is called.
74 OnStartLoadingResponseBody(handle<data_pipe_consumer> body); 73 OnStartLoadingResponseBody(handle<data_pipe_consumer> body);
75 74
76 // Called when the loading completes. No notification will be dispatched for 75 // Called when the loading completes. No notification will be dispatched for
77 // this client after this message arrives. 76 // this client after this message arrives.
78 OnComplete(URLLoaderStatus completion_status); 77 OnComplete(URLLoaderStatus completion_status);
79 }; 78 };
80 79
OLDNEW
« no previous file with comments | « content/child/url_loader_client_impl.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