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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: components/cronet/android/cronet_upload_data_stream_adapter.cc

Issue 2283243002: Allow direct executors in cronet. (Closed)
Patch Set: Created 4 years, 4 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
« components/cronet/android/api/src/org/chromium/net/UrlRequest.java ('K') | « components/cronet/android/api/src/org/chromium/net/UrlRequest.java ('k') | components/cronet/android/cronet_url_request_adapter.cc » ('j') | components/cronet/android/cronet_url_request_adapter.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/android/cronet_upload_data_stream_adapter.cc
diff --git a/components/cronet/android/cronet_upload_data_stream_adapter.cc b/components/cronet/android/cronet_upload_data_stream_adapter.cc
index af2854ba6c0a3bc911122aa7ba65866294d0183c..571c1c8a2d95049025858487b0ba5f7d351ead89 100644
--- a/components/cronet/android/cronet_upload_data_stream_adapter.cc
+++ b/components/cronet/android/cronet_upload_data_stream_adapter.cc
@@ -81,7 +81,6 @@ void CronetUploadDataStreamAdapter::OnReadSucceeded(
const JavaParamRef<jobject>& jcaller,
int bytes_read,
bool final_chunk) {
- DCHECK(!network_task_runner_->BelongsToCurrentThread());
DCHECK(bytes_read > 0 || (final_chunk && bytes_read == 0));
buffer_ = nullptr;
@@ -93,7 +92,6 @@ void CronetUploadDataStreamAdapter::OnReadSucceeded(
void CronetUploadDataStreamAdapter::OnRewindSucceeded(
JNIEnv* env,
const JavaParamRef<jobject>& jcaller) {
- DCHECK(!network_task_runner_->BelongsToCurrentThread());
network_task_runner_->PostTask(
FROM_HERE,
« components/cronet/android/api/src/org/chromium/net/UrlRequest.java ('K') | « components/cronet/android/api/src/org/chromium/net/UrlRequest.java ('k') | components/cronet/android/cronet_url_request_adapter.cc » ('j') | components/cronet/android/cronet_url_request_adapter.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698