Unify the QUIC client classes.

Unify the QUIC client classes.

Committed: https://crrev.com/ed113b20e8e8658be2bfc42faa093d8368e60900
Cr-Commit-Position: refs/heads/master@{#322323}

[Ryan Hamilton, 2015-03-25 22:56:22]

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

[ramant (doing other things), 2015-03-26 00:09:19]

Changes look ok to me. Shouldn't we delete
net/tools/quic/quic_simple_client_session_test.cc?

thanks
raman

https://codereview.chromium.org/1035533006/diff/1/net/tools/quic/quic_simple_...
File net/tools/quic/quic_simple_client_stream.h (left):

https://codereview.chromium.org/1035533006/diff/1/net/tools/quic/quic_simple_...
net/tools/quic/quic_simple_client_stream.h:29: class QuicSimpleClientStream :
public QuicDataStream {
Shouldn't we delete QuicSimpleClientStreamTest?

thanks
raman

https://code.google.com/p/chromium/codesearch#chromium/src/net/tools/quic/qui...

[Ryan Hamilton, 2015-03-26 00:33:36]

On 2015/03/26 00:09:19, ramant wrote:
> Changes look ok to me. Shouldn't we delete
> net/tools/quic/quic_simple_client_session_test.cc?
> 
> thanks
> raman
> 
>
https://codereview.chromium.org/1035533006/diff/1/net/tools/quic/quic_simple_...
> File net/tools/quic/quic_simple_client_stream.h (left):
> 
>
https://codereview.chromium.org/1035533006/diff/1/net/tools/quic/quic_simple_...
> net/tools/quic/quic_simple_client_stream.h:29: class QuicSimpleClientStream :
> public QuicDataStream {
> Shouldn't we delete QuicSimpleClientStreamTest?
> 
> thanks
> raman
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/tools/quic/qui...

Good point! Done.

[ramant (doing other things), 2015-03-26 00:35:49]

lgtm

Thanks much.

[Ryan Hamilton, 2015-03-26 00:48:50]

The CQ bit was checked by rch@chromium.org

[commit-bot: I haz the power, 2015-03-26 00:50:00]

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

[commit-bot: I haz the power, 2015-03-26 01:28:17]

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

[commit-bot: I haz the power, 2015-03-26 01:28:18]

Try jobs failed on following builders:
  win8_chromium_rel on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)

[Ryan Hamilton, 2015-03-26 03:35:08]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-03-26 03:35:08]

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/1035533006/#ps40001
(title: "ssize_t")

[commit-bot: I haz the power, 2015-03-26 03:36:02]

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

[commit-bot: I haz the power, 2015-03-26 03:40:27]

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

[commit-bot: I haz the power, 2015-03-26 03:40:30]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Ryan Hamilton, 2015-03-26 03:52:33]

Patchset #3 (id:40001) has been deleted

[Ryan Hamilton, 2015-03-26 03:52:58]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-03-26 03:52:59]

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/1035533006/#ps60001
(title: "ssize_t")

[commit-bot: I haz the power, 2015-03-26 03:53:25]

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

[commit-bot: I haz the power, 2015-03-26 04:54:13]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-03-26 04:55:29]

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