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

Side by Side Diff: components/cronet/android/cronet_bidirectional_stream_adapter.cc

Issue 1412243012: Initial implementation of CronetBidirectionalStream. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address Helen's comments. Created 4 years, 10 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
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "cronet_bidirectional_stream_adapter.h"
6
7 #include <string>
8 #include <vector>
9
10 #include "base/bind.h"
11 #include "base/location.h"
12 #include "base/logging.h"
13 #include "base/strings/string_number_conversions.h"
14 #include "components/cronet/android/cronet_url_request_context_adapter.h"
15 #include "jni/CronetBidirectionalStream_jni.h"
16 #include "net/base/io_buffer.h"
17 #include "net/base/net_errors.h"
18 #include "net/base/request_priority.h"
19 #include "net/http/bidirectional_stream_request_info.h"
20 #include "net/http/http_network_session.h"
21 #include "net/http/http_response_headers.h"
22 #include "net/http/http_status_code.h"
23 #include "net/http/http_transaction_factory.h"
24 #include "net/http/http_util.h"
25 #include "net/spdy/spdy_header_block.h"
26 #include "net/ssl/ssl_info.h"
27 #include "net/url_request/url_request_context.h"
28 #include "url/gurl.h"
29
30 using base::android::ConvertUTF8ToJavaString;
31 using base::android::ConvertJavaStringToUTF8;
32
33 namespace cronet {
34
35 static jlong CreateBidirectionalStream(
36 JNIEnv* env,
37 const JavaParamRef<jobject>& jbidi_stream,
38 jlong jurl_request_context_adapter) {
39 CronetURLRequestContextAdapter* context_adapter =
40 reinterpret_cast<CronetURLRequestContextAdapter*>(
41 jurl_request_context_adapter);
42 DCHECK(context_adapter);
43
44 CronetBidirectionalStreamAdapter* adapter =
45 new CronetBidirectionalStreamAdapter(context_adapter, env, jbidi_stream);
46
47 return reinterpret_cast<jlong>(adapter);
48 }
49
50 // TODO(mef): Extract this and its original from cronet_url_request_adapter.cc
51 // into separate module.
52 // net::WrappedIOBuffer subclass for a buffer owned by a Java ByteBuffer. Keeps
53 // the ByteBuffer alive until destroyed. Uses WrappedIOBuffer because data() is
54 // owned by the embedder.
55 class CronetBidirectionalStreamAdapter::IOBufferWithByteBuffer
56 : public net::WrappedIOBuffer {
57 public:
58 // Creates a buffer wrapping the Java ByteBuffer |jbyte_buffer|.
59 // |byte_buffer_data| points to the memory backed by the ByteBuffer, and
60 // |position| is the index of the first byte of data inside of the buffer.
61 // |limit| is the the index of the first element that should not be read or
62 // written, preserved to verify that buffer is not changed externally during
63 // networking operations.
64 IOBufferWithByteBuffer(JNIEnv* env,
65 const JavaParamRef<jobject>& jbyte_buffer,
66 void* byte_buffer_data,
67 int position,
68 int limit)
69 : net::WrappedIOBuffer(static_cast<char*>(byte_buffer_data) + position),
70 byte_buffer_(env, jbyte_buffer),
71 initial_position_(position),
72 initial_limit_(limit) {
73 DCHECK(byte_buffer_data);
74 DCHECK_EQ(env->GetDirectBufferAddress(jbyte_buffer), byte_buffer_data);
75 }
76
77 int initial_position() const { return initial_position_; }
78 int initial_limit() const { return initial_limit_; }
79
80 jobject byte_buffer() const { return byte_buffer_.obj(); }
81
82 private:
83 ~IOBufferWithByteBuffer() override {}
84
85 base::android::ScopedJavaGlobalRef<jobject> byte_buffer_;
86
87 const int initial_position_;
88 const int initial_limit_;
89 };
90
91 // static
92 bool CronetBidirectionalStreamAdapter::RegisterJni(JNIEnv* env) {
93 return RegisterNativesImpl(env);
94 }
95
96 CronetBidirectionalStreamAdapter::CronetBidirectionalStreamAdapter(
97 CronetURLRequestContextAdapter* context,
98 JNIEnv* env,
99 const JavaParamRef<jobject>& jbidi_stream)
100 : context_(context), owner_(env, jbidi_stream) {}
101
102 CronetBidirectionalStreamAdapter::~CronetBidirectionalStreamAdapter() {
103 DCHECK(context_->IsOnNetworkThread());
104 }
105
106 jint CronetBidirectionalStreamAdapter::Start(
107 JNIEnv* env,
108 const JavaParamRef<jobject>& jcaller,
109 const JavaParamRef<jstring>& jurl,
110 jint jpriority,
111 const JavaParamRef<jstring>& jmethod,
112 const JavaParamRef<jobjectArray>& jheaders,
113 jboolean jend_of_stream) {
114 // Prepare request info here to be able to return the error.
115 scoped_ptr<net::BidirectionalStreamRequestInfo> request_info(
116 new net::BidirectionalStreamRequestInfo());
117 request_info->url = GURL(ConvertJavaStringToUTF8(env, jurl));
118 request_info->priority = static_cast<net::RequestPriority>(jpriority);
119 // Http method is a token, just as header name.
120 request_info->method = ConvertJavaStringToUTF8(env, jmethod);
121 if (!net::HttpUtil::IsValidHeaderName(request_info->method))
122 return -1;
123
124 std::vector<std::string> headers;
125 base::android::AppendJavaStringArrayToStringVector(env, jheaders, &headers);
126 for (size_t i = 0; i < headers.size(); i += 2) {
127 std::string name(headers[i]);
128 std::string value(headers[i + 1]);
129 if (!net::HttpUtil::IsValidHeaderName(name) ||
130 !net::HttpUtil::IsValidHeaderValue(value)) {
131 return i + 1;
132 }
133 request_info->extra_headers.SetHeader(name, value);
134 }
135 request_info->end_stream_on_headers = jend_of_stream;
136
137 context_->PostTaskToNetworkThread(
138 FROM_HERE,
139 base::Bind(&CronetBidirectionalStreamAdapter::StartOnNetworkThread,
140 base::Unretained(this), base::Passed(&request_info)));
141 return 0;
142 }
143
144 jboolean CronetBidirectionalStreamAdapter::ReadData(
145 JNIEnv* env,
146 const JavaParamRef<jobject>& jcaller,
147 const JavaParamRef<jobject>& jbyte_buffer,
148 jint jposition,
149 jint jlimit) {
150 DCHECK_LT(jposition, jlimit);
151
152 void* data = env->GetDirectBufferAddress(jbyte_buffer);
153 if (!data)
154 return JNI_FALSE;
155
156 scoped_refptr<IOBufferWithByteBuffer> read_buffer(
157 new IOBufferWithByteBuffer(env, jbyte_buffer, data, jposition, jlimit));
158
159 int remaining_capacity = jlimit - jposition;
160
161 context_->PostTaskToNetworkThread(
162 FROM_HERE,
163 base::Bind(&CronetBidirectionalStreamAdapter::ReadDataOnNetworkThread,
164 base::Unretained(this), read_buffer, remaining_capacity));
165 return JNI_TRUE;
166 }
167
168 jboolean CronetBidirectionalStreamAdapter::WriteData(
169 JNIEnv* env,
170 const JavaParamRef<jobject>& jcaller,
171 const JavaParamRef<jobject>& jbyte_buffer,
172 jint jposition,
173 jint jlimit,
174 jboolean jend_of_stream) {
175 DCHECK_LE(jposition, jlimit);
176
177 void* data = env->GetDirectBufferAddress(jbyte_buffer);
178 if (!data)
179 return JNI_FALSE;
180
181 scoped_refptr<IOBufferWithByteBuffer> write_buffer(
182 new IOBufferWithByteBuffer(env, jbyte_buffer, data, jposition, jlimit));
183
184 int remaining_capacity = jlimit - jposition;
185
186 context_->PostTaskToNetworkThread(
187 FROM_HERE,
188 base::Bind(&CronetBidirectionalStreamAdapter::WriteDataOnNetworkThread,
189 base::Unretained(this), write_buffer, remaining_capacity,
190 jend_of_stream));
191 return JNI_TRUE;
192 }
193
194 void CronetBidirectionalStreamAdapter::Destroy(
195 JNIEnv* env,
196 const JavaParamRef<jobject>& jcaller,
197 jboolean jsend_on_canceled) {
198 // Destroy could be called from any thread, including network thread (if
199 // posting task to executor throws an exception), but is posted, so |this|
200 // is valid until calling task is complete. Destroy() is always called from
201 // within a synchronized java block that guarantees no future posts to the
202 // network thread with the adapter pointer.
203 context_->PostTaskToNetworkThread(
204 FROM_HERE,
205 base::Bind(&CronetBidirectionalStreamAdapter::DestroyOnNetworkThread,
206 base::Unretained(this), jsend_on_canceled));
207 }
208
209 void CronetBidirectionalStreamAdapter::OnHeadersSent() {
210 DCHECK(context_->IsOnNetworkThread());
211 JNIEnv* env = base::android::AttachCurrentThread();
212 cronet::Java_CronetBidirectionalStream_onRequestHeadersSent(env,
213 owner_.obj());
214 }
215
216 void CronetBidirectionalStreamAdapter::OnHeadersReceived(
217 const net::SpdyHeaderBlock& response_headers) {
218 DCHECK(context_->IsOnNetworkThread());
219 JNIEnv* env = base::android::AttachCurrentThread();
220 // Get http status code from response headers.
221 jint http_status_code = 0;
222 const auto http_status_header = response_headers.find(":status");
223 if (http_status_header != response_headers.end())
224 base::StringToInt(http_status_header->second, &http_status_code);
225
226 std::string protocol;
227 switch (bidi_stream_->GetProtocol()) {
228 case net::kProtoHTTP2:
229 protocol = "h2";
230 default:
231 break;
232 }
233
234 cronet::Java_CronetBidirectionalStream_onResponseHeadersReceived(
235 env, owner_.obj(), http_status_code,
236 ConvertUTF8ToJavaString(env, protocol).obj(),
237 GetHeadersArray(env, response_headers).obj(),
238 bidi_stream_->GetTotalReceivedBytes());
239 }
240
241 void CronetBidirectionalStreamAdapter::OnDataRead(int bytes_read) {
242 DCHECK(context_->IsOnNetworkThread());
243 JNIEnv* env = base::android::AttachCurrentThread();
244 cronet::Java_CronetBidirectionalStream_onReadCompleted(
245 env, owner_.obj(), read_buffer_->byte_buffer(), bytes_read,
246 read_buffer_->initial_position(), read_buffer_->initial_limit(),
247 bidi_stream_->GetTotalReceivedBytes());
248 // Free the read buffer. This lets the Java ByteBuffer be freed, if the
249 // embedder releases it, too.
250 read_buffer_ = nullptr;
251 }
252
253 void CronetBidirectionalStreamAdapter::OnDataSent() {
254 DCHECK(context_->IsOnNetworkThread());
255 JNIEnv* env = base::android::AttachCurrentThread();
256 cronet::Java_CronetBidirectionalStream_onWriteCompleted(
257 env, owner_.obj(), write_buffer_->byte_buffer(),
258 write_buffer_->initial_position(), write_buffer_->initial_limit());
259 // Free the write buffer. This lets the Java ByteBuffer be freed, if the
260 // embedder releases it, too.
261 write_buffer_ = nullptr;
262 }
263
264 void CronetBidirectionalStreamAdapter::OnTrailersReceived(
265 const net::SpdyHeaderBlock& response_trailers) {
266 DCHECK(context_->IsOnNetworkThread());
267 JNIEnv* env = base::android::AttachCurrentThread();
268 cronet::Java_CronetBidirectionalStream_onResponseTrailersReceived(
269 env, owner_.obj(), GetHeadersArray(env, response_trailers).obj());
270 }
271
272 void CronetBidirectionalStreamAdapter::OnFailed(int error) {
273 DCHECK(context_->IsOnNetworkThread());
274 JNIEnv* env = base::android::AttachCurrentThread();
275 cronet::Java_CronetBidirectionalStream_onError(
276 env, owner_.obj(), error,
277 ConvertUTF8ToJavaString(env, net::ErrorToString(error)).obj(),
278 bidi_stream_->GetTotalReceivedBytes());
279 }
280
281 void CronetBidirectionalStreamAdapter::StartOnNetworkThread(
282 scoped_ptr<net::BidirectionalStreamRequestInfo> request_info) {
283 DCHECK(context_->IsOnNetworkThread());
284 DCHECK(!bidi_stream_);
285 bidi_stream_.reset(new net::BidirectionalStream(
286 std::move(request_info), context_->GetURLRequestContext()
287 ->http_transaction_factory()
288 ->GetSession(),
289 this));
290 }
291
292 void CronetBidirectionalStreamAdapter::ReadDataOnNetworkThread(
293 scoped_refptr<IOBufferWithByteBuffer> read_buffer,
294 int buffer_size) {
295 DCHECK(context_->IsOnNetworkThread());
296 DCHECK(read_buffer);
297 DCHECK(!read_buffer_);
298
299 read_buffer_ = read_buffer;
300
301 int bytes_read = bidi_stream_->ReadData(read_buffer_.get(), buffer_size);
302 // If IO is pending, wait for the BidirectionalStream to call OnDataRead.
303 if (bytes_read == net::ERR_IO_PENDING)
304 return;
305
306 if (bytes_read < 0) {
307 OnFailed(bytes_read);
308 return;
309 }
310 OnDataRead(bytes_read);
311 }
312
313 void CronetBidirectionalStreamAdapter::WriteDataOnNetworkThread(
314 scoped_refptr<IOBufferWithByteBuffer> write_buffer,
315 int buffer_size,
316 bool end_of_stream) {
317 DCHECK(context_->IsOnNetworkThread());
318 DCHECK(write_buffer);
319 DCHECK(!write_buffer_);
320
321 write_buffer_ = write_buffer;
322 bidi_stream_->SendData(write_buffer_.get(), buffer_size, end_of_stream);
323 }
324
325 void CronetBidirectionalStreamAdapter::DestroyOnNetworkThread(
326 bool send_on_canceled) {
327 DCHECK(context_->IsOnNetworkThread());
328 if (send_on_canceled) {
329 JNIEnv* env = base::android::AttachCurrentThread();
330 cronet::Java_CronetBidirectionalStream_onCanceled(env, owner_.obj());
331 }
332 delete this;
333 }
334
335 base::android::ScopedJavaLocalRef<jobjectArray>
336 CronetBidirectionalStreamAdapter::GetHeadersArray(
337 JNIEnv* env,
338 const net::SpdyHeaderBlock& header_block) {
339 DCHECK(context_->IsOnNetworkThread());
340
341 std::vector<std::string> headers;
342 for (const auto& header : header_block) {
343 headers.push_back(header.first.as_string());
344 headers.push_back(header.second.as_string());
345 }
346 return base::android::ToJavaArrayOfStrings(env, headers);
347 }
348
349 } // namespace cronet
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698