Send encoded_body_length to renderer when response completed (2/3)

Send encoded_body_length to renderer when response completed (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

Design doc: https://docs.google.com/document/d/1Q-LpTPcV8QApeXSL24MVZHjP-uQK7RAK300t_qZaGFQ/edit#

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
Committed: https://crrev.com/b82cdb024079a83c15b4c2c02ea28700ef6797cf
Cr-Commit-Position: refs/heads/master@{#434654}

[yhirano, 2016-11-18 06:47:22]

The CQ bit was checked by yhirano@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-18 06:47:30]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-18 07:32:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-18 07:32:59]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[yhirano, 2016-11-18 07:34:01]

The CQ bit was checked by yhirano@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-18 07:34:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yhirano, 2016-11-18 07:40:47]

The CQ bit was checked by yhirano@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-18 07:40:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yhirano, 2016-11-18 07:59:32]

Patchset #2 (id:20001) has been deleted

[yhirano, 2016-11-18 07:59:40]

Patchset #1 (id:1) has been deleted

[yhirano, 2016-11-18 09:18:45]

Description was changed from

==========
Dispatch encoded_body_size at the completion timing (2/3)

BUG=666216
==========

to

==========
Dispatch encoded_body_length at the completion timing (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

[yhirano, 2016-11-18 09:18:45]

yhirano@chromium.org changed reviewers:
+ ricea@chromium.org, tzik@chromium.org

[commit-bot: I haz the power, 2016-11-18 11:42:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-18 11:42:35]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yhirano, 2016-11-21 02:31:59]

Description was changed from

==========
Dispatch encoded_body_length at the completion timing (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

to

==========
Dispatch encoded_body_length at the completion timing (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

[yhirano, 2016-11-21 06:39:08]

Description was changed from

==========
Dispatch encoded_body_length at the completion timing (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

to

==========
Send encoded_body_length to renderer when response completed (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

[Adam Rice, 2016-11-21 09:05:50]

lgtm

[yhirano, 2016-11-21 12:03:35]

Description was changed from

==========
Send encoded_body_length to renderer when response completed (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

to

==========
Send encoded_body_length to renderer when response completed (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

Design doc:
https://docs.google.com/document/d/1Q-LpTPcV8QApeXSL24MVZHjP-uQK7RAK300t_qZaG...

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

[kinuko, 2016-11-21 12:51:25]

lgtm

[tzik, 2016-11-22 05:54:09]

lgtm

[yhirano, 2016-11-22 07:37:12]

yhirano@chromium.org changed reviewers:
+ sky@chromium.org

[yhirano, 2016-11-22 07:37:13]

+sky@ for chrome/ changes.

[sky, 2016-11-22 20:07:02]

LGTM

[yhirano, 2016-11-28 09:14:37]

The CQ bit was checked by yhirano@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-28 09:14:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-28 10:58:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-28 10:58:06]

Dry run: This issue passed the CQ dry run.

[yhirano, 2016-11-28 14:13:20]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2016-11-28 14:13:20]

The patchset sent to the CQ was uploaded after l-g-t-m from kinuko@chromium.org,
ricea@chromium.org, sky@chromium.org, tzik@chromium.org
Link to the patchset: https://codereview.chromium.org/2510333002/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2016-11-28 14:13:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-28 14:17:46]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1480342400123330,
"parent_rev": "f79c30df4baa70dbd728672f92e8b1a96d7a7650", "commit_rev":
"82b7f20ca0c5937af3bad47fb40b64c5595b8e64"}

[commit-bot: I haz the power, 2016-11-28 14:18:08]

Description was changed from

==========
Send encoded_body_length to renderer when response completed (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

Design doc:
https://docs.google.com/document/d/1Q-LpTPcV8QApeXSL24MVZHjP-uQK7RAK300t_qZaG...

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

to

==========
Send encoded_body_length to renderer when response completed (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

Design doc:
https://docs.google.com/document/d/1Q-LpTPcV8QApeXSL24MVZHjP-uQK7RAK300t_qZaG...

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

[commit-bot: I haz the power, 2016-11-28 14:18:10]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-11-28 14:20:15]

Description was changed from

==========
Send encoded_body_length to renderer when response completed (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

Design doc:
https://docs.google.com/document/d/1Q-LpTPcV8QApeXSL24MVZHjP-uQK7RAK300t_qZaG...

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216
==========

to

==========
Send encoded_body_length to renderer when response completed (2/3)

Currently, "encoded_body_length" is dispatched on each ResourceMsg_DataReceived
IPC message. With this series of CLs, it will be dispatched when a resource
loading is completed. This is needed to use mojo data pipe for transferring
the response body.

 - 1/3: https://codereview.chromium.org/2514713002/
 - 2/3: https://codereview.chromium.org/2510333002/
 - 3/3: https://codereview.chromium.org/2506363005/

Design doc:
https://docs.google.com/document/d/1Q-LpTPcV8QApeXSL24MVZHjP-uQK7RAK300t_qZaG...

This CL
 - removes encoded_body_length() from RequestPeer::ReceivedData.
 - removes encodedBodyLength parameter from WebURLLoaderClient::didReceiveData,
 - adds totalEncodedBodyLength parameter to
WebURLLoaderClient::didFinishLoading,
 - adds totalEncodedBodyLength parameter to WebURLLoaderClient::didFail.

BUG=666216

Committed: https://crrev.com/b82cdb024079a83c15b4c2c02ea28700ef6797cf
Cr-Commit-Position: refs/heads/master@{#434654}
==========

[commit-bot: I haz the power, 2016-11-28 14:20:16]

Patchset 2 (id:??) landed as
https://crrev.com/b82cdb024079a83c15b4c2c02ea28700ef6797cf
Cr-Commit-Position: refs/heads/master@{#434654}