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

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

Issue 2566943002: Dispatch transfer size update notification on mojo-loading (Closed)
Patch Set: fix Created 4 years 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 26 matching lines...) Expand all
37 // the 'download_to_file' case. 37 // the 'download_to_file' case.
38 OnReceiveResponse(URLResponseHead head, DownloadedTempFile? downloaded_file); 38 OnReceiveResponse(URLResponseHead head, DownloadedTempFile? downloaded_file);
39 39
40 // 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
41 // call FollowRedirect or cancel the request. 41 // call FollowRedirect or cancel the request.
42 OnReceiveRedirect(URLRequestRedirectInfo redirect_info, URLResponseHead head); 42 OnReceiveRedirect(URLRequestRedirectInfo redirect_info, URLResponseHead head);
43 43
44 // 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
45 // 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
46 // OnStartLoadingResponseBody in the call sequence in that case. 46 // OnStartLoadingResponseBody in the call sequence in that case.
47 // TODO(yhirano): Remove |encoded_length| and use OnTransferSizeUpdated
48 // instead.
47 OnDataDownloaded(int64 data_length, int64 encoded_length); 49 OnDataDownloaded(int64 data_length, int64 encoded_length);
48 50
51 // Called when the transfer size is updated. This is only called if
52 // |report_raw_headers| is set and |download_to_file| is unset in the request.
53 // The transfer size is the length of the response (including both headers
54 // and the body) over the network. |transfer_size_diff| is the difference from
55 // the value previously reported one (including the one in OnReceiveResponse
56 // and OnReceiveRedirect). It must be positive.
57 // TODO(yhirano): Dispatch this notification even when |download_to_file| is
58 // set.
59 // TODO(yhirano): Consider using an unsigned type.
60 OnTransferSizeUpdated(int32 transfer_size_diff);
61
49 // Called when the loader starts loading response body. 62 // Called when the loader starts loading response body.
50 OnStartLoadingResponseBody(handle<data_pipe_consumer> body); 63 OnStartLoadingResponseBody(handle<data_pipe_consumer> body);
51 64
52 // Called when the loading completes. No notification will be dispatched for 65 // Called when the loading completes. No notification will be dispatched for
53 // this client after this message arrives. 66 // this client after this message arrives.
54 OnComplete(URLLoaderStatus completion_status); 67 OnComplete(URLLoaderStatus completion_status);
55 }; 68 };
56 69
OLDNEW
« 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