Fix a pair of Cronet upload bugs and add tests.

net/base/net_error_list.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file intentionally does not have header guards, it's included
 // inside a macro to generate enum values.
 
 // This file contains the list of network errors.
 
 //
@@ skipping 81 matching lines @@
 NET_ERROR(BLOCKED_BY_ADMINISTRATOR, -22)
 
 // The socket is already connected.
 NET_ERROR(SOCKET_IS_CONNECTED, -23)
 
 // The request was blocked because the forced reenrollment check is still
 // pending. This error can only occur on ChromeOS.
 // The error can be emitted by code in chrome/browser/policy/policy_helpers.cc.
 NET_ERROR(BLOCKED_ENROLLMENT_CHECK_PENDING, -24)
 
+// The upload failed because the upload stream needed to be re-read, due to a
+// retry or a redirect, but the upload stream doesn't support that operation.
+NET_ERROR(UPLOAD_STREAM_REWIND_NOT_SUPPORTED, -25)

[xunjieli, 2014/10/17 20:56:45]

Is it possible to test that we get this error code?

[mmenke, 2014/10/17 21:27:15]

I don't believe we expose it with the current API - we should with the new one,
but I don't think it's worth adding much more to the old one.

+
 // A connection was closed (corresponding to a TCP FIN).
 NET_ERROR(CONNECTION_CLOSED, -100)
 
 // A connection was reset (corresponding to a TCP RST).
 NET_ERROR(CONNECTION_RESET, -101)
 
 // A connection attempt was refused.
 NET_ERROR(CONNECTION_REFUSED, -102)
 
 // A connection timed out as a result of not receiving an ACK for data sent.
@@ skipping 657 matching lines @@
 NET_ERROR(DNS_TIMED_OUT, -803)
 
 // The entry was not found in cache, for cache-only lookups.
 NET_ERROR(DNS_CACHE_MISS, -804)
 
 // Suffix search list rules prevent resolution of the given host name.
 NET_ERROR(DNS_SEARCH_EMPTY, -805)
 
 // Failed to sort addresses according to RFC3484.
 NET_ERROR(DNS_SORT_ERROR, -806)