Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(738)

Unified Diff: net/quic/chromium/bidirectional_stream_quic_impl.h

Issue 2908243002: Remove QuicChromiumClientStream::Delegate in favor of async methods. (Closed)
Patch Set: Rebase Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/quic/chromium/bidirectional_stream_quic_impl.h
diff --git a/net/quic/chromium/bidirectional_stream_quic_impl.h b/net/quic/chromium/bidirectional_stream_quic_impl.h
index 3f20ee13a9db3bee54d3b85d34306d5f3b062568..55ea0d0e6628619468ffd9e431d9186a9b266db3 100644
--- a/net/quic/chromium/bidirectional_stream_quic_impl.h
+++ b/net/quic/chromium/bidirectional_stream_quic_impl.h
@@ -29,8 +29,7 @@ struct BidirectionalStreamRequestInfo;
class IOBuffer;
class NET_EXPORT_PRIVATE BidirectionalStreamQuicImpl
- : public BidirectionalStreamImpl,
- public QuicChromiumClientStream::Delegate {
+ : public BidirectionalStreamImpl {
public:
explicit BidirectionalStreamQuicImpl(
std::unique_ptr<QuicChromiumClientSession::Handle> session);
@@ -57,10 +56,6 @@ class NET_EXPORT_PRIVATE BidirectionalStreamQuicImpl
bool GetLoadTimingInfo(LoadTimingInfo* load_timing_info) const override;
private:
- // QuicChromiumClientStream::Delegate implementation:
- void OnClose() override;
- void OnError(int error) override;
-
bool WriteHeaders();
void OnStreamReady(int rv);
void OnSendDataComplete(int rv);
@@ -94,6 +89,7 @@ class NET_EXPORT_PRIVATE BidirectionalStreamQuicImpl
LoadTimingInfo::ConnectTiming connect_timing_;
SpdyHeaderBlock initial_headers_;
+ bool expect_trailers_; // True if trailers are expected to arrive.
SpdyHeaderBlock trailing_headers_;
// User provided read buffer for ReadData() response.

Powered by Google App Engine
This is Rietveld 408576698