Use net::UploadDataStream::Init() instead of InitSync()

Use net::UploadDataStream::Init() instead of InitSync()

BUG=113669
TEST=net_unittests


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

[hashimoto, 2012-08-06 08:48:37]

This change depends on http://codereview.chromium.org/10823180/

[hashimoto, 2012-08-08 03:29:54]

Eric, could you review this change?

[eroman, 2012-08-08 19:06:01]

lgtm

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

http://codereview.chromium.org/10829188/diff/1/net/http/http_network_transact...
net/http/http_network_transaction.cc:755: int error_code = OK;
nit: for consistency you may want to call this  |rv|.

http://codereview.chromium.org/10829188/diff/1/net/http/http_network_transact...
net/http/http_network_transaction.cc:760: request_body_.reset(NULL);
You can remove this: it will already be reset by DoInitRequestBodyComplete()

[hashimoto, 2012-08-09 04:35:20]

Thank you for reviewing.
Could you review http://codereview.chromium.org/10823180/ too?
This change depends on that.

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

http://codereview.chromium.org/10829188/diff/1/net/http/http_network_transact...
net/http/http_network_transaction.cc:755: int error_code = OK;
On 2012/08/08 19:06:01, eroman wrote:
> nit: for consistency you may want to call this  |rv|.

Done.

http://codereview.chromium.org/10829188/diff/1/net/http/http_network_transact...
net/http/http_network_transaction.cc:760: request_body_.reset(NULL);
On 2012/08/08 19:06:01, eroman wrote:
> You can remove this: it will already be reset by DoInitRequestBodyComplete()

Done.

[commit-bot: I haz the power, 2012-09-19 04:01:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hashimoto@chromium.org/10829188/15001

[commit-bot: I haz the power, 2012-09-19 05:51:14]

Change committed as 157512