Port bunch of files in net/

Port bunch of files in net/

BUG=2623

[dank, 2008-10-23 15:56:36]

LGTM, with comments.

http://codereview.chromium.org/7916/diff/1/2
File net/http/http_network_transaction.cc (right):

http://codereview.chromium.org/7916/diff/1/2#newcode424
Line 424: if ((reused_socket_ = (connection_.socket() != NULL))) {
Better to split the assignment off into its own
statement...?

http://codereview.chromium.org/7916/diff/1/3
File net/net_unittests.scons (right):

http://codereview.chromium.org/7916/diff/1/3#newcode109
Line 109: 'http/http_network_transaction_unittest.cc',
You forgot to enable the tests for one of the files you fixed:
    'http/http_transaction_unittest.cc',
    'http/http_network_transaction_unittest.cc',
They build and pass on linux with your patch.

[please use my chromium address, 2008-10-23 16:07:40]

http://codereview.chromium.org/7916/diff/1/2
File net/http/http_network_transaction.cc (right):

http://codereview.chromium.org/7916/diff/1/2#newcode424
Line 424: if ((reused_socket_ = (connection_.socket() != NULL))) {
On 2008/10/23 15:56:36, dank wrote:
> Better to split the assignment off into its own
> statement...?

Done.

http://codereview.chromium.org/7916/diff/1/3
File net/net_unittests.scons (right):

http://codereview.chromium.org/7916/diff/1/3#newcode109
Line 109: 'http/http_network_transaction_unittest.cc',
On 2008/10/23 15:56:36, dank wrote:
> You forgot to enable the tests for one of the files you fixed:
>     'http/http_transaction_unittest.cc',
>     'http/http_network_transaction_unittest.cc',
> They build and pass on linux with your patch.

Done, thanks.

[Mark Mentovai, 2008-10-23 16:18:44]

LGTM

[Mark Mentovai, 2008-10-23 21:42:30]

Checked in, r3867