relnote: change ExtractContentLengthFromHeaders to return false if content-length is missing. If th…

relnote: change ExtractContentLengthFromHeaders to return false if content-length is missing. If the caller is tolerate to have content-length header missing, it should call ExtractContentLengthFromHeaders only when content-length header is present.

Merge internal change: 135027159

BUG=

Committed: https://crrev.com/34b14f3281e01845d9ef7cf5512855a417798370
Cr-Commit-Position: refs/heads/master@{#422642}

[Zhongyi Shi, 2016-10-03 22:13:29]

Description was changed from

==========
relnote: change ExtractContentLengthFromHeaders to return false if
content-length is missing. If the caller is tolerate to have content-length
header missing, it should call ExtractContentLengthFromHeaders only when
content-length header is present.

Merge internal change: 135027159

BUG=
==========

to

==========
relnote: change ExtractContentLengthFromHeaders to return false if
content-length is missing. If the caller is tolerate to have content-length
header missing, it should call ExtractContentLengthFromHeaders only when
content-length header is present.

Merge internal change: 135027159

BUG=
==========

[Zhongyi Shi, 2016-10-03 22:13:29]

zhongyi@chromium.org changed reviewers:
+ rch@chromium.org

[Ryan Hamilton, 2016-10-03 22:15:42]

lgtm

[Zhongyi Shi, 2016-10-03 22:16:02]

The CQ bit was checked by zhongyi@chromium.org

[commit-bot: I haz the power, 2016-10-03 22:16:50]

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-10-03 22:54:13]

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

[commit-bot: I haz the power, 2016-10-03 22:54:14]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Zhongyi Shi, 2016-10-03 23:10:20]

The CQ bit was checked by zhongyi@chromium.org

[Zhongyi Shi, 2016-10-03 23:10:21]

The patchset sent to the CQ was uploaded after l-g-t-m from rch@chromium.org
Link to the patchset: https://codereview.chromium.org/2385323004/#ps20001
(title: "fix test")

[Zhongyi Shi, 2016-10-03 23:10:36]

The CQ bit was unchecked by zhongyi@chromium.org

[Zhongyi Shi, 2016-10-03 23:10:50]

The CQ bit was checked by zhongyi@chromium.org

[commit-bot: I haz the power, 2016-10-03 23:11:34]

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-10-04 00:53:37]

Description was changed from

==========
relnote: change ExtractContentLengthFromHeaders to return false if
content-length is missing. If the caller is tolerate to have content-length
header missing, it should call ExtractContentLengthFromHeaders only when
content-length header is present.

Merge internal change: 135027159

BUG=
==========

to

==========
relnote: change ExtractContentLengthFromHeaders to return false if
content-length is missing. If the caller is tolerate to have content-length
header missing, it should call ExtractContentLengthFromHeaders only when
content-length header is present.

Merge internal change: 135027159

BUG=
==========

[commit-bot: I haz the power, 2016-10-04 00:53:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-04 00:55:21]

Description was changed from

==========
relnote: change ExtractContentLengthFromHeaders to return false if
content-length is missing. If the caller is tolerate to have content-length
header missing, it should call ExtractContentLengthFromHeaders only when
content-length header is present.

Merge internal change: 135027159

BUG=
==========

to

==========
relnote: change ExtractContentLengthFromHeaders to return false if
content-length is missing. If the caller is tolerate to have content-length
header missing, it should call ExtractContentLengthFromHeaders only when
content-length header is present.

Merge internal change: 135027159

BUG=

Committed: https://crrev.com/34b14f3281e01845d9ef7cf5512855a417798370
Cr-Commit-Position: refs/heads/master@{#422642}
==========

[commit-bot: I haz the power, 2016-10-04 00:55:22]

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