parseHTTPHeaders should return the number of consumed bytes.

parseHTTPHeaders should return the number of consumed bytes.

Currently parseHTTPHeaders returns (the number of consumed bytes - x) where
 - x = 0 if it consumes an empty line
 - x = 1 otherwise
This CL make parseHTTPHeaders always return the number of consumed bytes.

BUG=NONE
R=tyoshino

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175630

[tyoshino (SeeGerritForStatus), 2014-06-05 08:24:15]

https://codereview.chromium.org/314083002/diff/1/Source/platform/network/HTTP...
File Source/platform/network/HTTPRequest.cpp (right):

https://codereview.chromium.org/314083002/diff/1/Source/platform/network/HTTP...
Source/platform/network/HTTPRequest.cpp:90: break;
we never reached here before your last fix?

[yhirano, 2014-06-05 08:33:19]

https://codereview.chromium.org/314083002/diff/1/Source/platform/network/HTTP...
File Source/platform/network/HTTPRequest.cpp (right):

https://codereview.chromium.org/314083002/diff/1/Source/platform/network/HTTP...
Source/platform/network/HTTPRequest.cpp:90: break;
On 2014/06/05 08:24:15, tyoshino wrote:
> we never reached here before your last fix?

Yes, when "\r\n" is consumed.

[tyoshino (SeeGerritForStatus), 2014-06-05 08:37:33]

lgtm

[yhirano, 2014-06-05 08:38:15]

+tkent for OWNER review.

[tkent, 2014-06-05 23:03:43]

lgtm

[commit-bot: I haz the power, 2014-06-05 23:04:12]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yhirano@chromium.org/314083002/1

[commit-bot: I haz the power, 2014-06-06 00:22:21]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/10490)
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[commit-bot: I haz the power, 2014-06-06 00:22:43]

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

[commit-bot: I haz the power, 2014-06-06 00:22:43]

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[tkent, 2014-06-06 00:28:29]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2014-06-06 00:28:58]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yhirano@chromium.org/314083002/1

[commit-bot: I haz the power, 2014-06-06 01:29:43]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)

[commit-bot: I haz the power, 2014-06-06 01:32:27]

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

[commit-bot: I haz the power, 2014-06-06 01:32:28]

Try jobs failed on following builders:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)

[yhirano, 2014-06-06 01:34:15]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2014-06-06 01:35:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yhirano@chromium.org/314083002/1

[commit-bot: I haz the power, 2014-06-06 03:27:51]

Change committed as 175630