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

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: self review Created 4 years, 8 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
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 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 class CronetBidirectionalStreamAdapter 70 class CronetBidirectionalStreamAdapter
71 : public cronet::CronetBidirectionalStream::Delegate { 71 : public cronet::CronetBidirectionalStream::Delegate {
72 public: 72 public:
73 CronetBidirectionalStreamAdapter( 73 CronetBidirectionalStreamAdapter(
74 cronet_engine* engine, 74 cronet_engine* engine,
75 cronet_bidirectional_stream* stream, 75 cronet_bidirectional_stream* stream,
76 cronet_bidirectional_stream_callback* callback); 76 cronet_bidirectional_stream_callback* callback);
77 77
78 virtual ~CronetBidirectionalStreamAdapter(); 78 virtual ~CronetBidirectionalStreamAdapter();
79 79
80 void OnHeadersSent() override; 80 void OnStreamReady() override;
81 81
82 void OnHeadersReceived(const net::SpdyHeaderBlock& headers_block, 82 void OnHeadersReceived(const net::SpdyHeaderBlock& headers_block,
83 const char* negotiated_protocol) override; 83 const char* negotiated_protocol) override;
84 84
85 void OnDataRead(char* data, int size) override; 85 void OnDataRead(char* data, int size) override;
86 86
87 void OnDataSent(const char* data) override; 87 void OnDataSent(const char* data) override;
88 88
89 void OnTrailersReceived(const net::SpdyHeaderBlock& trailers_block) override; 89 void OnTrailersReceived(const net::SpdyHeaderBlock& trailers_block) override;
90 90
(...skipping 27 matching lines...) Expand all
118 reinterpret_cast<cronet::CronetEnvironment*>(engine->obj)), 118 reinterpret_cast<cronet::CronetEnvironment*>(engine->obj)),
119 stream_(stream), 119 stream_(stream),
120 callback_(callback) { 120 callback_(callback) {
121 DCHECK(cronet_environment_); 121 DCHECK(cronet_environment_);
122 cronet_bidirectional_stream_ = 122 cronet_bidirectional_stream_ =
123 new cronet::CronetBidirectionalStream(cronet_environment_, this); 123 new cronet::CronetBidirectionalStream(cronet_environment_, this);
124 } 124 }
125 125
126 CronetBidirectionalStreamAdapter::~CronetBidirectionalStreamAdapter() {} 126 CronetBidirectionalStreamAdapter::~CronetBidirectionalStreamAdapter() {}
127 127
128 void CronetBidirectionalStreamAdapter::OnHeadersSent() { 128 void CronetBidirectionalStreamAdapter::OnStreamReady() {
129 DCHECK(callback_->on_response_headers_received); 129 DCHECK(callback_->on_response_headers_received);
130 callback_->on_request_headers_sent(stream_); 130 callback_->on_stream_ready(stream_);
131 } 131 }
132 132
133 void CronetBidirectionalStreamAdapter::OnHeadersReceived( 133 void CronetBidirectionalStreamAdapter::OnHeadersReceived(
134 const net::SpdyHeaderBlock& headers_block, 134 const net::SpdyHeaderBlock& headers_block,
135 const char* negotiated_protocol) { 135 const char* negotiated_protocol) {
136 DCHECK(callback_->on_response_headers_received); 136 DCHECK(callback_->on_response_headers_received);
137 HeadersArray response_headers(headers_block); 137 HeadersArray response_headers(headers_block);
138 callback_->on_response_headers_received(stream_, &response_headers, 138 callback_->on_response_headers_received(stream_, &response_headers,
139 negotiated_protocol); 139 negotiated_protocol);
140 } 140 }
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
261 int cronet_bidirectional_stream_cancel(cronet_bidirectional_stream* stream) { 261 int cronet_bidirectional_stream_cancel(cronet_bidirectional_stream* stream) {
262 CronetBidirectionalStreamAdapter::GetCronetStream(stream)->Cancel(); 262 CronetBidirectionalStreamAdapter::GetCronetStream(stream)->Cancel();
263 return 1; 263 return 1;
264 } 264 }
265 265
266 int cronet_bidirectional_stream_destroy(cronet_bidirectional_stream* stream) { 266 int cronet_bidirectional_stream_destroy(cronet_bidirectional_stream* stream) {
267 CronetBidirectionalStreamAdapter::DestroyAdapterForStream(stream); 267 CronetBidirectionalStreamAdapter::DestroyAdapterForStream(stream);
268 delete stream; 268 delete stream;
269 return 1; 269 return 1;
270 } 270 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698