Remove Balsa from QuicSpdyClientStream.

Remove Balsa from QuicSpdyClientStream.

To facilitate landing actual QuicServer support in Chromium using
Chromium's MessageLoop and friends.

Merge internal change: 89526581

Committed: https://crrev.com/40fa174740fe5c16d4e1377bad45791206b8bae7
Cr-Commit-Position: refs/heads/master@{#322246}

[Ryan Hamilton, 2015-03-25 20:54:16]

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

[ramant (doing other things), 2015-03-25 21:06:22]

lgtm

https://codereview.chromium.org/1006193003/diff/20001/net/tools/quic/quic_spd...
File net/tools/quic/quic_spdy_client_stream.cc (right):

https://codereview.chromium.org/1006193003/diff/20001/net/tools/quic/quic_spd...
net/tools/quic/quic_spdy_client_stream.cc:5: 
nit: extra blank line.

https://codereview.chromium.org/1006193003/diff/20001/net/tools/quic/test_too...
File net/tools/quic/test_tools/quic_test_client.cc (right):

https://codereview.chromium.org/1006193003/diff/20001/net/tools/quic/test_too...
net/tools/quic/test_tools/quic_test_client.cc:470: } else {
overly nit: could "return ... } else {" be deleted?

if (stream_ != nullptr) {
  SpdyUtils::SpdyHeadersToResponseHeaders(stream_->headers(), &headers_);
}
return &headers_;

[Ryan Hamilton, 2015-03-25 21:37:43]

Thanks!

https://codereview.chromium.org/1006193003/diff/20001/net/tools/quic/quic_spd...
File net/tools/quic/quic_spdy_client_stream.cc (right):

https://codereview.chromium.org/1006193003/diff/20001/net/tools/quic/quic_spd...
net/tools/quic/quic_spdy_client_stream.cc:5: 
On 2015/03/25 21:06:21, ramant wrote:
> nit: extra blank line.

Done.

https://codereview.chromium.org/1006193003/diff/20001/net/tools/quic/test_too...
File net/tools/quic/test_tools/quic_test_client.cc (right):

https://codereview.chromium.org/1006193003/diff/20001/net/tools/quic/test_too...
net/tools/quic/test_tools/quic_test_client.cc:470: } else {
On 2015/03/25 21:06:21, ramant wrote:
> overly nit: could "return ... } else {" be deleted?
> 
> if (stream_ != nullptr) {
>   SpdyUtils::SpdyHeadersToResponseHeaders(stream_->headers(), &headers_);
> }
> return &headers_;

Done.

[Ryan Hamilton, 2015-03-25 21:37:47]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-03-25 21:37:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
rtenneti@chromium.org
Link to the patchset: https://codereview.chromium.org/1006193003/#ps40001
(title: "fix comments")

[commit-bot: I haz the power, 2015-03-25 21:38:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1006193003/40001

[commit-bot: I haz the power, 2015-03-25 22:21:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-25 22:21:51]

Patchset 3 (id:??) landed as
https://crrev.com/40fa174740fe5c16d4e1377bad45791206b8bae7
Cr-Commit-Position: refs/heads/master@{#322246}