Update ResourceResponse::encodedDataLength when response is completed

Update ResourceResponse::encodedDataLength when response is completed

This CL updates ResourceResponse::encodedDataLength when a response is completed
either successfully or unsuccessfully.

This is a preliminary change to stop dispatching encoded_data_length on each
data chunk arrival.

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

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

[yhirano, 2016-11-21 08:19:50]

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

[commit-bot: I haz the power, 2016-11-21 08:19:58]

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-21 08:35:56]

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

[commit-bot: I haz the power, 2016-11-21 08:35:57]

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

[yhirano, 2016-11-21 11:24:07]

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

[commit-bot: I haz the power, 2016-11-21 11:24:20]

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-21 12:25:34]

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

[commit-bot: I haz the power, 2016-11-21 12:25:37]

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-21 12:36:24]

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

[commit-bot: I haz the power, 2016-11-21 12:36:36]

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-21 13:59:24]

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

[commit-bot: I haz the power, 2016-11-21 13:59:25]

Dry run: This issue passed the CQ dry run.

[yhirano, 2016-11-22 01:36:59]

Patchset #1 (id:1) has been deleted

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

Patchset #1 (id:20001) has been deleted

[yhirano, 2016-11-22 01:37:19]

Patchset #1 (id:40001) has been deleted

[yhirano, 2016-11-22 01:38:34]

Description was changed from

==========
Update ResourceResponse::encodedDataLength when response completes

This CL updates ResourceResponse::encodedDataLength when a response completes
either successfully or unsuccessfully.

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

to

==========
Update ResourceResponse::encodedDataLength when response completes

This CL updates ResourceResponse::encodedDataLength when a response completes
either successfully or unsuccessfully.

This is a preliminary change to stop dispatching encoded_data_length on each
data chunk arrival.

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

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

[yhirano, 2016-11-22 01:38:34]

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

[yhirano, 2016-11-22 01:39:24]

hiroshige@, tzik@: PTAL

cc: allada@ fyi

[kinuko, 2016-11-22 03:35:46]

lgtm once/if other reviewers are happy

[tzik, 2016-11-22 06:53:58]

lgtm

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

Description was changed from

==========
Update ResourceResponse::encodedDataLength when response completes

This CL updates ResourceResponse::encodedDataLength when a response completes
either successfully or unsuccessfully.

This is a preliminary change to stop dispatching encoded_data_length on each
data chunk arrival.

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

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

to

==========
Update ResourceResponse::encodedDataLength when response is completed

This CL updates ResourceResponse::encodedDataLength when a response is completed
either successfully or unsuccessfully.

This is a preliminary change to stop dispatching encoded_data_length on each
data chunk arrival.

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

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

[hiroshige, 2016-11-22 08:26:07]

https://codereview.chromium.org/2519843002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/fetch/Resource.h (right):

https://codereview.chromium.org/2519843002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/fetch/Resource.h:303: void
setEncodedDataLength(int value) {
Shouldn't this |int64_t value|?

[yhirano, 2016-11-22 08:42:41]

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

[commit-bot: I haz the power, 2016-11-22 08:42:50]

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-22 08:43:42]

https://codereview.chromium.org/2519843002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/fetch/Resource.h (right):

https://codereview.chromium.org/2519843002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/fetch/Resource.h:303: void
setEncodedDataLength(int value) {
On 2016/11/22 08:26:06, hiroshige wrote:
> Shouldn't this |int64_t value|?

Thanks, you're right.

[commit-bot: I haz the power, 2016-11-22 10:25:52]

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

[commit-bot: I haz the power, 2016-11-22 10:25:52]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-11-22 11:01:24]

lgtm.

[yhirano, 2016-11-22 11:08:50]

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

[commit-bot: I haz the power, 2016-11-22 11:08:57]

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-22 11:09:33]

The CQ bit was unchecked by yhirano@chromium.org

[yhirano, 2016-11-22 11:09:34]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2016-11-22 11:09:34]

The patchset sent to the CQ was uploaded after l-g-t-m from kinuko@chromium.org,
tzik@chromium.org, hiroshige@chromium.org
Link to the patchset: https://codereview.chromium.org/2519843002/#ps100001
(title: "Update ResourceResponse::encodedDataLength when response is completed")

[commit-bot: I haz the power, 2016-11-22 11:09:44]

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-22 14:56:38]

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

[commit-bot: I haz the power, 2016-11-22 14:56:39]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[yhirano, 2016-11-22 15:52:23]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2016-11-22 15:52:42]

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-22 17:08:37]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1479829943009090,
"parent_rev": "d3017bc263365228769ce4f3610e963d11289c1e", "commit_rev":
"2b765735677b80c81e393542bf6b28943a5d7a53"}

[commit-bot: I haz the power, 2016-11-22 17:09:10]

Description was changed from

==========
Update ResourceResponse::encodedDataLength when response is completed

This CL updates ResourceResponse::encodedDataLength when a response is completed
either successfully or unsuccessfully.

This is a preliminary change to stop dispatching encoded_data_length on each
data chunk arrival.

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

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

to

==========
Update ResourceResponse::encodedDataLength when response is completed

This CL updates ResourceResponse::encodedDataLength when a response is completed
either successfully or unsuccessfully.

This is a preliminary change to stop dispatching encoded_data_length on each
data chunk arrival.

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

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

[commit-bot: I haz the power, 2016-11-22 17:09:12]

Committed patchset #3 (id:100001)

[commit-bot: I haz the power, 2016-11-22 17:12:16]

Description was changed from

==========
Update ResourceResponse::encodedDataLength when response is completed

This CL updates ResourceResponse::encodedDataLength when a response is completed
either successfully or unsuccessfully.

This is a preliminary change to stop dispatching encoded_data_length on each
data chunk arrival.

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

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

to

==========
Update ResourceResponse::encodedDataLength when response is completed

This CL updates ResourceResponse::encodedDataLength when a response is completed
either successfully or unsuccessfully.

This is a preliminary change to stop dispatching encoded_data_length on each
data chunk arrival.

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

BUG=666216

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

[commit-bot: I haz the power, 2016-11-22 17:12:18]

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