sandwich: Handle 200 responses with no data.

sandwich: Handle 200 responses with no data.

Before sandwich was asserting that the request encoded_data_length > 0.
But it turn out that encoded_data_length does not include the response
headers' length. As a result, this assertion was failing on some website
doing XHR requests that were returning 200 response.

This CL fixes this issue by correctly computing the total number of
byte transfered back to chrome with GetResponseTransportLength(),
including the response headers size.

BUG=582080
Committed: https://crrev.com/2dad1bfcf999e58230aabcba1e75b5e994b31fcc
Cr-Commit-Position: refs/heads/master@{#404415}

[gabadie, 2016-07-08 13:42:23]

gabadie@chromium.org changed reviewers:
+ pasko@chromium.org

[pasko, 2016-07-08 15:08:08]

lgtm, thank you

[gabadie, 2016-07-08 15:18:48]

The CQ bit was checked by gabadie@chromium.org

[gabadie, 2016-07-08 15:18:49]

The patchset sent to the CQ was uploaded after l-g-t-m from pasko@chromium.org
Link to the patchset: https://codereview.chromium.org/2131973002/#ps20001
(title: "Rebase for landing")

[commit-bot: I haz the power, 2016-07-08 15:19:05]

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

[commit-bot: I haz the power, 2016-07-08 15:19:07]

This CL has an open dependency (Issue 2134763002 Patch 1). Please resolve the
dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[gabadie, 2016-07-08 15:24:41]

The CQ bit was checked by gabadie@chromium.org

[gabadie, 2016-07-08 15:24:42]

The patchset sent to the CQ was uploaded after l-g-t-m from pasko@chromium.org
Link to the patchset: https://codereview.chromium.org/2131973002/#ps40001
(title: "Remove unfortunate dependency")

[commit-bot: I haz the power, 2016-07-08 15:24:54]

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-07-08 16:49:01]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-08 16:52:06]

Description was changed from

==========
sandwich: Handle 200 responses with no data.

Before sandwich was asserting that the request encoded_data_length > 0.
But it turn out that encoded_data_length does not include the response
headers' length. As a result, this assertion was failing on some website
doing XHR requests that were returning 200 response.

This CL fixes this issue by correctly computing the total number of
byte transfered back to chrome with GetResponseTransportLength(),
including the response headers size.

BUG=582080
==========

to

==========
sandwich: Handle 200 responses with no data.

Before sandwich was asserting that the request encoded_data_length > 0.
But it turn out that encoded_data_length does not include the response
headers' length. As a result, this assertion was failing on some website
doing XHR requests that were returning 200 response.

This CL fixes this issue by correctly computing the total number of
byte transfered back to chrome with GetResponseTransportLength(),
including the response headers size.

BUG=582080

Committed: https://crrev.com/2dad1bfcf999e58230aabcba1e75b5e994b31fcc
Cr-Commit-Position: refs/heads/master@{#404415}
==========

[commit-bot: I haz the power, 2016-07-08 16:52:07]

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