[Cronet] Implement batch read in CronetInputStream

components/cronet/android/cronet_url_request.cc

 // Copyright 2014 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.
 
 #include "components/cronet/android/cronet_url_request.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/logging.h"
@@ skipping 267 matching lines @@
 
 static jstring GetHttpStatusText(JNIEnv* env,
                                  jobject jurl_request,
                                  jlong jurl_request_adapter) {
   DCHECK(jurl_request_adapter);
   CronetURLRequestAdapter* request_adapter =
       reinterpret_cast<CronetURLRequestAdapter*>(jurl_request_adapter);
   DCHECK(request_adapter->IsOnNetworkThread());
   const net::HttpResponseHeaders* headers =
       request_adapter->GetResponseHeaders();
+  if (headers == NULL)
+    return ConvertUTF8ToJavaString(env, "").Release();

[xunjieli, 2015/03/04 18:10:53]

This might not seem very relevant, but it is needed for the test to not crash
(See: crbug.com/463720 the mock data job does not give back headers). So I
though I'd add the null check in this CL. I'll separate it out into a new CL, if
you do not like it.

[mmenke, 2015/03/05 19:24:46]

Should we add a test just for this?

[xunjieli, 2015/03/05 20:55:13]

Talked to Matt offline. The mock job triggered this null case. Not very easy to
trigger otherwise.

   return ConvertUTF8ToJavaString(env, headers->GetStatusText()).Release();
 }
 
 }  // namespace cronet