Strip content-encoding response header from CronetHttpURLConnection response.

Strip content-encoding response header from CronetHttpURLConnection response.

Cronet decompresses response body before returning it to consumers, we should
strip Content-Encoding headers so that consumers wouldn't try to decompress for
a second time.

BUG=592700
Committed: https://crrev.com/cee2840c4bcd51313e33f0ab215413f9a8f70480
Cr-Commit-Position: refs/heads/master@{#379903}

[xunjieli, 2016-03-08 16:49:07]

Description was changed from

==========
Strip content-encoding response header from CronetHttpURLConnection response.
o
Cronet decompresses response body before returning it to consumers, we should
strip Content-Encoding headers so that consumers wouldn't try to decompress for
a second time.

BUG=592700
==========

to

==========
Strip content-encoding response header from CronetHttpURLConnection response.

Cronet decompresses response body before returning it to consumers, we should
strip Content-Encoding headers so that consumers wouldn't try to decompress for
a second time.

BUG=592700
==========

[xunjieli, 2016-03-08 16:50:42]

xunjieli@chromium.org changed reviewers:
+ kapishnikov@chromium.org

[xunjieli, 2016-03-08 16:50:42]

Andrei, PTAL.

[kapishnikov, 2016-03-08 19:37:17]

LGTM

https://codereview.chromium.org/1779473002/diff/1/components/cronet/android/j...
File
components/cronet/android/java/src/org/chromium/net/urlconnection/CronetHttpURLConnection.java
(right):

https://codereview.chromium.org/1779473002/diff/1/components/cronet/android/j...
components/cronet/android/java/src/org/chromium/net/urlconnection/CronetHttpURLConnection.java:565:
// TODO(xunjieli): Refactor to reuse code in UrlRequestInfo.
Did you mean UrlResponseInfo?

[xunjieli, 2016-03-08 19:51:14]

thanks!

https://codereview.chromium.org/1779473002/diff/1/components/cronet/android/j...
File
components/cronet/android/java/src/org/chromium/net/urlconnection/CronetHttpURLConnection.java
(right):

https://codereview.chromium.org/1779473002/diff/1/components/cronet/android/j...
components/cronet/android/java/src/org/chromium/net/urlconnection/CronetHttpURLConnection.java:565:
// TODO(xunjieli): Refactor to reuse code in UrlRequestInfo.
On 2016/03/08 19:37:17, kapishnikov wrote:
> Did you mean UrlResponseInfo?

Done.

[xunjieli, 2016-03-08 19:51:18]

The CQ bit was checked by xunjieli@chromium.org

[xunjieli, 2016-03-08 19:51:18]

The patchset sent to the CQ was uploaded after l-g-t-m from
kapishnikov@chromium.org
Link to the patchset: https://codereview.chromium.org/1779473002/#ps20001
(title: "Address comment")

[commit-bot: I haz the power, 2016-03-08 19:51:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1779473002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1779473002/20001

[commit-bot: I haz the power, 2016-03-08 20:44:08]

Description was changed from

==========
Strip content-encoding response header from CronetHttpURLConnection response.

Cronet decompresses response body before returning it to consumers, we should
strip Content-Encoding headers so that consumers wouldn't try to decompress for
a second time.

BUG=592700
==========

to

==========
Strip content-encoding response header from CronetHttpURLConnection response.

Cronet decompresses response body before returning it to consumers, we should
strip Content-Encoding headers so that consumers wouldn't try to decompress for
a second time.

BUG=592700
==========

[commit-bot: I haz the power, 2016-03-08 20:44:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-08 20:45:21]

Description was changed from

==========
Strip content-encoding response header from CronetHttpURLConnection response.

Cronet decompresses response body before returning it to consumers, we should
strip Content-Encoding headers so that consumers wouldn't try to decompress for
a second time.

BUG=592700
==========

to

==========
Strip content-encoding response header from CronetHttpURLConnection response.

Cronet decompresses response body before returning it to consumers, we should
strip Content-Encoding headers so that consumers wouldn't try to decompress for
a second time.

BUG=592700

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

[commit-bot: I haz the power, 2016-03-08 20:45:21]

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