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

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

Issue 2561743003: Use associated interface on DownloadedTempFile (Closed)
Patch Set: +NOTREACHED on non frame trasfer 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, DownloadedTempFile? downloaded_file); 38 OnReceiveResponse(URLResponseHead head,
39 associated DownloadedTempFile? downloaded_file);
39 40
40 // Called when the request has been redirected. The receiver is expected to 41 // Called when the request has been redirected. The receiver is expected to
41 // call FollowRedirect or cancel the request. 42 // call FollowRedirect or cancel the request.
42 OnReceiveRedirect(URLRequestRedirectInfo redirect_info, URLResponseHead head); 43 OnReceiveRedirect(URLRequestRedirectInfo redirect_info, URLResponseHead head);
43 44
44 // Called when some data from a resource request has been downloaded to the 45 // Called when some data from a resource request has been downloaded to the
45 // file. This is only called in the 'download_to_file' case and replaces 46 // file. This is only called in the 'download_to_file' case and replaces
46 // OnStartLoadingResponseBody in the call sequence in that case. 47 // OnStartLoadingResponseBody in the call sequence in that case.
47 // TODO(yhirano): Remove |encoded_length| and use OnTransferSizeUpdated 48 // TODO(yhirano): Remove |encoded_length| and use OnTransferSizeUpdated
48 // instead. 49 // instead.
(...skipping 15 matching lines...) Expand all
64 65
65 // Called when the loader starts loading response body. This is called after 66 // Called when the loader starts loading response body. This is called after
66 // OnReceiveResponse is called. 67 // OnReceiveResponse is called.
67 OnStartLoadingResponseBody(handle<data_pipe_consumer> body); 68 OnStartLoadingResponseBody(handle<data_pipe_consumer> body);
68 69
69 // Called when the loading completes. No notification will be dispatched for 70 // Called when the loading completes. No notification will be dispatched for
70 // this client after this message arrives. 71 // this client after this message arrives.
71 OnComplete(URLLoaderStatus completion_status); 72 OnComplete(URLLoaderStatus completion_status);
72 }; 73 };
73 74
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