Unify the QUIC client classes.

net/tools/quic/quic_simple_client_stream.h

Index: net/tools/quic/quic_simple_client_stream.h
diff --git a/net/tools/quic/quic_simple_client_stream.h b/net/tools/quic/quic_simple_client_stream.h
deleted file mode 100644
index b4669a8de74cae12fe0cc2a490b40403c0fad033..0000000000000000000000000000000000000000
--- a/net/tools/quic/quic_simple_client_stream.h
+++ /dev/null
@@ -1,92 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#ifndef NET_TOOLS_QUIC_QUIC_SIMPLE_CLIENT_STREAM_H_
-#define NET_TOOLS_QUIC_QUIC_SIMPLE_CLIENT_STREAM_H_
-
-#include <sys/types.h>
-#include <string>
-
-#include "base/basictypes.h"
-#include "base/strings/string_piece.h"
-#include "net/base/io_buffer.h"
-#include "net/http/http_request_info.h"
-#include "net/http/http_response_headers.h"
-#include "net/quic/quic_data_stream.h"
-#include "net/quic/quic_protocol.h"
-#include "net/tools/balsa/balsa_frame.h"
-#include "net/tools/balsa/balsa_headers.h"
-
-namespace net {
-
-namespace tools {
-
-class QuicSimpleClientSession;
-
-// All this does right now is send an SPDY request, and aggregate the
-// SPDY response.
-class QuicSimpleClientStream : public QuicDataStream {

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

Shouldn't we delete QuicSimpleClientStreamTest?

thanks
raman

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

- public:
-  QuicSimpleClientStream(QuicStreamId id, QuicSimpleClientSession* session);
-  ~QuicSimpleClientStream() override;
-
-  // Override the base class to close the write side as soon as we get a
-  // response.
-  // SPDY/HTTP does not support bidirectional streaming.
-  void OnStreamFrame(const QuicStreamFrame& frame) override;
-
-  // Override the base class to store the size of the headers.
-  void OnStreamHeadersComplete(bool fin, size_t frame_len) override;
-
-  // ReliableQuicStream implementation called by the session when there's
-  // data for us.
-  uint32 ProcessData(const char* data, uint32 data_len) override;
-
-  void OnFinRead() override;
-
-  // Serializes the headers and body, sends it to the server, and
-  // returns the number of bytes sent.
-  size_t SendRequest(const HttpRequestInfo& headers,
-                     base::StringPiece body,
-                     bool fin);
-
-  // Sends body data to the server, or buffers if it can't be sent immediately.
-  void SendBody(const std::string& data, bool fin);
-  // As above, but |delegate| will be notified once |data| is ACKed.
-  void SendBody(const std::string& data,
-                bool fin,
-                QuicAckNotifier::DelegateInterface* delegate);
-
-  // Returns the response data.
-  const std::string& data() { return data_; }
-
-  // Returns whatever headers have been received for this stream.
-  scoped_refptr<HttpResponseHeaders> headers() { return headers_; }
-
-  size_t header_bytes_read() const { return header_bytes_read_; }
-
-  size_t header_bytes_written() const { return header_bytes_written_; }
-
-  // While the server's set_priority shouldn't be called externally, the creator
-  // of client-side streams should be able to set the priority.
-  using QuicDataStream::set_priority;
-
- private:
-  int ParseResponseHeaders();
-
-  scoped_refptr<HttpResponseHeaders> headers_;
-  std::string data_;
-
-  scoped_refptr<GrowableIOBuffer> read_buf_;
-  bool response_headers_received_;
-  size_t header_bytes_read_;
-  size_t header_bytes_written_;
-
-  DISALLOW_COPY_AND_ASSIGN(QuicSimpleClientStream);
-};
-
-}  // namespace tools
-}  // namespace net
-
-#endif  // NET_TOOLS_QUIC_QUIC_SIMPLE_CLIENT_STREAM_H_