Remove any use of BalsaHeaders and HttpMessage from QuicTestClient.

Remove any use of BalsaHeaders and HttpMessage from QuicTestClient.

Merge internal change: 137947300

BUG=

Committed: https://crrev.com/2fec850438d6e4595e5fe255749dfdb8e91cc34d
Cr-Commit-Position: refs/heads/master@{#429615}

[Ryan Hamilton, 2016-11-03 12:16:01]

The CQ bit was checked by rch@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-03 12:16:18]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ryan Hamilton, 2016-11-03 12:39:49]

The CQ bit was checked by rch@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-03 12:39:55]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ryan Hamilton, 2016-11-03 12:49:43]

rch@chromium.org changed reviewers:
+ zhongyi@chromium.org

[commit-bot: I haz the power, 2016-11-03 13:13:13]

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

[commit-bot: I haz the power, 2016-11-03 13:13:14]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[Zhongyi Shi, 2016-11-03 15:15:22]

https://codereview.chromium.org/2470353002/diff/40001/net/tools/quic/end_to_e...
File net/tools/quic/end_to_end_test.cc (right):

https://codereview.chromium.org/2470353002/diff/40001/net/tools/quic/end_to_e...
net/tools/quic/end_to_end_test.cc:1243: client_->SendMessage(headers, body);
I think this should be:
client_->SendCustomSynchronousRequest(headers, body);

https://codereview.chromium.org/2470353002/diff/40001/net/tools/quic/end_to_e...
net/tools/quic/end_to_end_test.cc:1271: client_->SendMessage(headers,
large_body);
ditto

[Ryan Hamilton, 2016-11-03 15:22:20]

The CQ bit was checked by rch@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-03 15:22:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ryan Hamilton, 2016-11-03 15:25:52]

The CQ bit was checked by rch@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-03 15:26:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ryan Hamilton, 2016-11-03 15:26:18]

https://codereview.chromium.org/2470353002/diff/40001/net/tools/quic/end_to_e...
File net/tools/quic/end_to_end_test.cc (right):

https://codereview.chromium.org/2470353002/diff/40001/net/tools/quic/end_to_e...
net/tools/quic/end_to_end_test.cc:1243: client_->SendMessage(headers, body);
On 2016/11/03 15:15:22, Zhongyi Shi wrote:
> I think this should be:
> client_->SendCustomSynchronousRequest(headers, body);

Done.

https://codereview.chromium.org/2470353002/diff/40001/net/tools/quic/end_to_e...
net/tools/quic/end_to_end_test.cc:1271: client_->SendMessage(headers,
large_body);
On 2016/11/03 15:15:22, Zhongyi Shi wrote:
> ditto

Done.

[Zhongyi Shi, 2016-11-03 15:43:47]

lgtm

[Ryan Hamilton, 2016-11-03 16:06:54]

Description was changed from

==========
Remove any use of BalsaHeaders and HttpMessage from QuicTestClient.

Merge internal change: 137845073

BUG=
==========

to

==========
Remove any use of BalsaHeaders and HttpMessage from QuicTestClient.

Merge internal change: 137947300

BUG=
==========

[Ryan Hamilton, 2016-11-03 16:07:09]

The CQ bit was unchecked by rch@chromium.org

[Ryan Hamilton, 2016-11-03 16:07:26]

The CQ bit was checked by rch@chromium.org

[commit-bot: I haz the power, 2016-11-03 16:07:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-03 16:35:20]

Description was changed from

==========
Remove any use of BalsaHeaders and HttpMessage from QuicTestClient.

Merge internal change: 137947300

BUG=
==========

to

==========
Remove any use of BalsaHeaders and HttpMessage from QuicTestClient.

Merge internal change: 137947300

BUG=
==========

[commit-bot: I haz the power, 2016-11-03 16:35:22]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-11-03 16:43:39]

Description was changed from

==========
Remove any use of BalsaHeaders and HttpMessage from QuicTestClient.

Merge internal change: 137947300

BUG=
==========

to

==========
Remove any use of BalsaHeaders and HttpMessage from QuicTestClient.

Merge internal change: 137947300

BUG=

Committed: https://crrev.com/2fec850438d6e4595e5fe255749dfdb8e91cc34d
Cr-Commit-Position: refs/heads/master@{#429615}
==========

[commit-bot: I haz the power, 2016-11-03 16:43:40]

Patchset 5 (id:??) landed as
https://crrev.com/2fec850438d6e4595e5fe255749dfdb8e91cc34d
Cr-Commit-Position: refs/heads/master@{#429615}