Handle HTTP-Version information as per RFC2616 sec 3.1 references.

Handle HTTP-Version information as per RFC2616 sec 3.1 references.

As per RFC2616 sec 3.1: HTTP-Version = "HTTP" "/" 1*DIGIT "." 1*DIGIT
Previously 1*DIGIT only handle 1 digit,
And now logic implemented for one or more digits as specified in TODO comments.

BUG=382120

[payal.pandey, 2015-05-07 10:47:44]

payal.pandey@samsung.com changed reviewers:
+ eroman@chromium.org

[payal.pandey, 2015-05-07 10:47:45]

Please have a look, Thanks.

[payal.pandey, 2015-05-07 11:17:21]

.

[eroman, 2015-05-08 02:54:56]

not lgtm:

  * Needs test cases
  * Should test for error conditions. For instance StringToInt() returns wether
was successful
  * Should not truncate the result on underflow
  * Variable naming should not use camelCase.

You should also familiarize yourself with the past attempt I rejected to do
this:

https://codereview.chromium.org/527883002/

Or consider a bug which has a more measurable improvement (this is more of a
nice-to-have, but not at a risk of hitting this as a compatibility problem in
the forseeable future)

[payal.pandey, 2015-05-20 13:45:57]

Please have a look, again.
Thanks

[eroman, 2015-05-20 22:16:16]

I didn't review in depth, but it looks better. A few quick comments:

 * The return value of StringToUint is not checked, please do so

 *  Please also add a test that exceeds 2^64 (I see you have one for 2^32
already)

 * Please add more tests with leading 0s. In particular:
     A number which would be interpreted differently if read as octal vs decimal
(in other words anything larger than 8)
     Also a number with multiple leading zeros

* Don't build a std::string that copies the intermediary number. The rest of the
file is using iterators to avoid making copies of the data. Please match that
style here.

[payal.pandey, 2015-05-21 07:41:02]

Updated as per review comments except leading zeros.
As this CL is only targeting 1*More digits in http version, not leading zeros.

Will upload leading zeros in new CL after this CL acceptance.

Please have a look, again.
Thanks.

[payal.pandey, 2015-05-23 19:26:22]

Please review the updated CL