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

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 d65f1a93ba97b2aeadf5e7678138f8c04d78cae8..07902830bb8a2a862234a7c7ad2407f34523e7db 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,14 +56,7 @@ 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;
-
- // Write headers to the stream and returns true on success. Posts a task to
- // notify the delegate asynchronously and returns false on failure
- bool WriteHeaders();
-
+ int WriteHeaders();
void OnStreamReady(int rv);
void OnSendDataComplete(int rv);
void ReadInitialHeaders();
@@ -105,6 +97,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