New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 874152 link

Starred by 1 user
Status: Fixed
Owner:
█ xunji...@chromium.org
Last visit > 30 days ago
Closed: Aug 15
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Task



Sign in to add a comment

Update CronetBufferedOutputStream to provide better error msg

Project Member Reported by xunji...@chromium.org, Aug 14 
Instead of saying "java.lang.IllegalStateException: Cannot write after being connected," it can prompt the consumer to set fixed streaming mode or chunked mode.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 15 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/104b778b131d430ca6bd9b1756b1589d86ba6cf6

commit 104b778b131d430ca6bd9b1756b1589d86ba6cf6
Author: Helen Li <xunjieli@chromium.org>
Date: Wed Aug 15 14:11:35 2018

Improve error msg in CronetBufferedOutputStream

Bug:  874152 
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester
Change-Id: I45441e5fdd8eaa4b0e39144cd94d610d3f98f368
Reviewed-on: https://chromium-review.googlesource.com/1174935
Commit-Queue: Paul Jensen <pauljensen@chromium.org>
Reviewed-by: Paul Jensen <pauljensen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583240}
[modify] https://crrev.com/104b778b131d430ca6bd9b1756b1589d86ba6cf6/components/cronet/android/java/src/org/chromium/net/urlconnection/CronetBufferedOutputStream.java
[modify] https://crrev.com/104b778b131d430ca6bd9b1756b1589d86ba6cf6/components/cronet/android/test/javatests/src/org/chromium/net/urlconnection/CronetBufferedOutputStreamTest.java

Comment 2 by xunji...@chromium.org, Aug 15

Status: Fixed (was: Assigned)
►

Sign in to add a comment