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

Side by Side Diff: components/cronet/ios/cronet_c_for_grpc.cc

Issue 1856073002: Coalesce small buffers in net::BidirectionalStream (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix javadoc Created 4 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 unified diff | Download patch
« no previous file with comments | « components/cronet/ios/cronet_c_for_grpc.h ('k') | net/http/bidirectional_stream.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/cronet/ios/cronet_c_for_grpc.h" 5 #include "components/cronet/ios/cronet_c_for_grpc.h"
6 6
7 #include <stdbool.h> 7 #include <stdbool.h>
8 8
9 #include <memory> 9 #include <memory>
10 #include <string> 10 #include <string>
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 class CronetBidirectionalStreamAdapter 71 class CronetBidirectionalStreamAdapter
72 : public cronet::CronetBidirectionalStream::Delegate { 72 : public cronet::CronetBidirectionalStream::Delegate {
73 public: 73 public:
74 CronetBidirectionalStreamAdapter( 74 CronetBidirectionalStreamAdapter(
75 cronet_engine* engine, 75 cronet_engine* engine,
76 cronet_bidirectional_stream* stream, 76 cronet_bidirectional_stream* stream,
77 cronet_bidirectional_stream_callback* callback); 77 cronet_bidirectional_stream_callback* callback);
78 78
79 virtual ~CronetBidirectionalStreamAdapter(); 79 virtual ~CronetBidirectionalStreamAdapter();
80 80
81 void OnHeadersSent() override; 81 void OnStreamReady() override;
82 82
83 void OnHeadersReceived(const net::SpdyHeaderBlock& headers_block, 83 void OnHeadersReceived(const net::SpdyHeaderBlock& headers_block,
84 const char* negotiated_protocol) override; 84 const char* negotiated_protocol) override;
85 85
86 void OnDataRead(char* data, int size) override; 86 void OnDataRead(char* data, int size) override;
87 87
88 void OnDataSent(const char* data) override; 88 void OnDataSent(const char* data) override;
89 89
90 void OnTrailersReceived(const net::SpdyHeaderBlock& trailers_block) override; 90 void OnTrailersReceived(const net::SpdyHeaderBlock& trailers_block) override;
91 91
(...skipping 27 matching lines...) Expand all
119 reinterpret_cast<cronet::CronetEnvironment*>(engine->obj)), 119 reinterpret_cast<cronet::CronetEnvironment*>(engine->obj)),
120 stream_(stream), 120 stream_(stream),
121 callback_(callback) { 121 callback_(callback) {
122 DCHECK(cronet_environment_); 122 DCHECK(cronet_environment_);
123 cronet_bidirectional_stream_ = 123 cronet_bidirectional_stream_ =
124 new cronet::CronetBidirectionalStream(cronet_environment_, this); 124 new cronet::CronetBidirectionalStream(cronet_environment_, this);
125 } 125 }
126 126
127 CronetBidirectionalStreamAdapter::~CronetBidirectionalStreamAdapter() {} 127 CronetBidirectionalStreamAdapter::~CronetBidirectionalStreamAdapter() {}
128 128
129 void CronetBidirectionalStreamAdapter::OnHeadersSent() { 129 void CronetBidirectionalStreamAdapter::OnStreamReady() {
130 DCHECK(callback_->on_response_headers_received); 130 DCHECK(callback_->on_response_headers_received);
131 callback_->on_request_headers_sent(stream_); 131 callback_->on_stream_ready(stream_);
132 } 132 }
133 133
134 void CronetBidirectionalStreamAdapter::OnHeadersReceived( 134 void CronetBidirectionalStreamAdapter::OnHeadersReceived(
135 const net::SpdyHeaderBlock& headers_block, 135 const net::SpdyHeaderBlock& headers_block,
136 const char* negotiated_protocol) { 136 const char* negotiated_protocol) {
137 DCHECK(callback_->on_response_headers_received); 137 DCHECK(callback_->on_response_headers_received);
138 HeadersArray response_headers(headers_block); 138 HeadersArray response_headers(headers_block);
139 callback_->on_response_headers_received(stream_, &response_headers, 139 callback_->on_response_headers_received(stream_, &response_headers,
140 negotiated_protocol); 140 negotiated_protocol);
141 } 141 }
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
262 int cronet_bidirectional_stream_cancel(cronet_bidirectional_stream* stream) { 262 int cronet_bidirectional_stream_cancel(cronet_bidirectional_stream* stream) {
263 CronetBidirectionalStreamAdapter::GetCronetStream(stream)->Cancel(); 263 CronetBidirectionalStreamAdapter::GetCronetStream(stream)->Cancel();
264 return 1; 264 return 1;
265 } 265 }
266 266
267 int cronet_bidirectional_stream_destroy(cronet_bidirectional_stream* stream) { 267 int cronet_bidirectional_stream_destroy(cronet_bidirectional_stream* stream) {
268 CronetBidirectionalStreamAdapter::DestroyAdapterForStream(stream); 268 CronetBidirectionalStreamAdapter::DestroyAdapterForStream(stream);
269 delete stream; 269 delete stream;
270 return 1; 270 return 1;
271 } 271 }
OLDNEW
« no previous file with comments | « components/cronet/ios/cronet_c_for_grpc.h ('k') | net/http/bidirectional_stream.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698