Treat all 1xx the same as a 100 (continue).

Treat all 1xx the same as a 100 (continue).

BUG=8440
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=12634

[wtc, 2009-03-26 21:00:40]

LGTM.

http://codereview.chromium.org/53111/diff/1/3
File net/http/http_network_transaction_unittest.cc (right):

http://codereview.chromium.org/53111/diff/1/3#newcode558
Line 558: TEST_F(HttpNetworkTransactionTest, Ignores1xx) {
You may want to point out how this unit test differs from
the next (Ignores100).  I believe the only difference is
the response status line ("102 Unspecified status code" vs.
"100 Continue".

[eroman, 2009-03-26 21:07:25]

http://codereview.chromium.org/53111/diff/1/3
File net/http/http_network_transaction_unittest.cc (right):

http://codereview.chromium.org/53111/diff/1/3#newcode558
Line 558: TEST_F(HttpNetworkTransactionTest, Ignores1xx) {
On 2009/03/26 21:00:40, wtc wrote:
> You may want to point out how this unit test differs from
> the next (Ignores100).  I believe the only difference is
> the response status line ("102 Unspecified status code" vs.
> "100 Continue".

The other minor difference is I specify HTTP/1.1 in the response, rather than
Ignores100 which specifies HTTP/1.0.

(technically http/1.0 does NOT define 1xx status codes... good to have tests
that differ a bit though)

[eroman, 2009-03-26 21:24:44]

http://codereview.chromium.org/53111/diff/1/3
File net/http/http_network_transaction_unittest.cc (right):

http://codereview.chromium.org/53111/diff/1/3#newcode558
Line 558: TEST_F(HttpNetworkTransactionTest, Ignores1xx) {
On 2009/03/26 21:00:40, wtc wrote:
> You may want to point out how this unit test differs from
> the next (Ignores100).  I believe the only difference is
> the response status line ("102 Unspecified status code" vs.
> "100 Continue".

Done.

Also moved this to appear below the Ignores100 test.