Commit e42d8cb0 authored by xunjieli's avatar xunjieli Committed by Commit bot

[Cronet] Move PostTask out of DCHECK.

DCHECK is removed in release builds, so we should move PostTask out of DCHECK.

Review URL: https://codereview.chromium.org/947203002

Cr-Commit-Position: refs/heads/master@{#317601}
parent 95a5170e
...@@ -79,9 +79,9 @@ void CronetUploadDataStreamDelegate::OnReadSucceeded(JNIEnv* env, ...@@ -79,9 +79,9 @@ void CronetUploadDataStreamDelegate::OnReadSucceeded(JNIEnv* env,
DCHECK(bytes_read > 0 || (final_chunk && bytes_read == 0)); DCHECK(bytes_read > 0 || (final_chunk && bytes_read == 0));
buffer_ = nullptr; buffer_ = nullptr;
DCHECK(network_task_runner_->PostTask( network_task_runner_->PostTask(
FROM_HERE, base::Bind(&CronetUploadDataStreamAdapter::OnReadSuccess, FROM_HERE, base::Bind(&CronetUploadDataStreamAdapter::OnReadSuccess,
adapter_, bytes_read, final_chunk))); adapter_, bytes_read, final_chunk));
} }
void CronetUploadDataStreamDelegate::OnRewindSucceeded(JNIEnv* env, void CronetUploadDataStreamDelegate::OnRewindSucceeded(JNIEnv* env,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment