Revert of Send encoded_body_length to renderer when response completed (1/3)

Revert of Send encoded_body_length to renderer when response completed (1/3) (patchset #4 id:60001 of https://codereview.chromium.org/2514713002/ )

Reason for revert:
I believe this change broke cronet tests:

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Marshmallow%2064bit%20Perf/builds/3490

Original issue's description:
> Send encoded_body_length to renderer when response completed (1/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 adds "encoded_body_length" property to ResourceRequestCompletionStatus.
> This CL doesn't change the behavior.
>
> BUG=666216
>
> Committed: https://crrev.com/1033246a67ce26d0150242a071e61330be318433
> Cr-Commit-Position: refs/heads/master@{#434032}

TBR=ricea@chromium.org,tzik@chromium.org,rdsmith@chromium.org,tsepez@chromium.org,yhirano@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=666216
Committed: https://crrev.com/b1bc4b42b14c2f238186b04920a504dc8bb41d2d
Cr-Commit-Position: refs/heads/master@{#434064}

[Maria, 2016-11-23 01:04:23]

The CQ bit was checked by mariakhomenko@chromium.org

[Maria, 2016-11-23 01:04:23]

Created Revert of Send encoded_body_length to renderer when response completed
(1/3)

[commit-bot: I haz the power, 2016-11-23 01:05:05]

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-23 01:16:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479863063014600, "parent_rev":
"6baee263c6a7ac61ed64b05c66d984f9ad09d9a4", "commit_rev":
"55e33376ea4141cbb6b6d432721f18a13c8090a3"}

[commit-bot: I haz the power, 2016-11-23 01:17:33]

Description was changed from

==========
Revert of Send encoded_body_length to renderer when response completed (1/3)
(patchset #4 id:60001 of https://codereview.chromium.org/2514713002/ )

Reason for revert:
I believe this change broke cronet tests:

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Mar...

Original issue's description:
> Send encoded_body_length to renderer when response completed (1/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 adds "encoded_body_length" property to
ResourceRequestCompletionStatus.
> This CL doesn't change the behavior.
> 
> BUG=666216
> 
> Committed: https://crrev.com/1033246a67ce26d0150242a071e61330be318433
> Cr-Commit-Position: refs/heads/master@{#434032}

TBR=ricea@chromium.org,tzik@chromium.org,rdsmith@chromium.org,tsepez@chromium...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=666216
==========

to

==========
Revert of Send encoded_body_length to renderer when response completed (1/3)
(patchset #4 id:60001 of https://codereview.chromium.org/2514713002/ )

Reason for revert:
I believe this change broke cronet tests:

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Mar...

Original issue's description:
> Send encoded_body_length to renderer when response completed (1/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 adds "encoded_body_length" property to
ResourceRequestCompletionStatus.
> This CL doesn't change the behavior.
> 
> BUG=666216
> 
> Committed: https://crrev.com/1033246a67ce26d0150242a071e61330be318433
> Cr-Commit-Position: refs/heads/master@{#434032}

TBR=ricea@chromium.org,tzik@chromium.org,rdsmith@chromium.org,tsepez@chromium...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=666216
==========

[commit-bot: I haz the power, 2016-11-23 01:17:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-23 01:19:58]

Description was changed from

==========
Revert of Send encoded_body_length to renderer when response completed (1/3)
(patchset #4 id:60001 of https://codereview.chromium.org/2514713002/ )

Reason for revert:
I believe this change broke cronet tests:

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Mar...

Original issue's description:
> Send encoded_body_length to renderer when response completed (1/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 adds "encoded_body_length" property to
ResourceRequestCompletionStatus.
> This CL doesn't change the behavior.
> 
> BUG=666216
> 
> Committed: https://crrev.com/1033246a67ce26d0150242a071e61330be318433
> Cr-Commit-Position: refs/heads/master@{#434032}

TBR=ricea@chromium.org,tzik@chromium.org,rdsmith@chromium.org,tsepez@chromium...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=666216
==========

to

==========
Revert of Send encoded_body_length to renderer when response completed (1/3)
(patchset #4 id:60001 of https://codereview.chromium.org/2514713002/ )

Reason for revert:
I believe this change broke cronet tests:

https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Mar...

Original issue's description:
> Send encoded_body_length to renderer when response completed (1/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 adds "encoded_body_length" property to
ResourceRequestCompletionStatus.
> This CL doesn't change the behavior.
>
> BUG=666216
>
> Committed: https://crrev.com/1033246a67ce26d0150242a071e61330be318433
> Cr-Commit-Position: refs/heads/master@{#434032}

TBR=ricea@chromium.org,tzik@chromium.org,rdsmith@chromium.org,tsepez@chromium...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=666216

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

[commit-bot: I haz the power, 2016-11-23 01:19:59]

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