[Cronet] Implement batch read in CronetInputStream

components/cronet/android/cronet_url_request.cc

Index: components/cronet/android/cronet_url_request.cc
diff --git a/components/cronet/android/cronet_url_request.cc b/components/cronet/android/cronet_url_request.cc
index bc7b29fea313a72e6d5a580c8669b23e6f1647d0..04489bfce73fe7c108f2c4ee7517424f609f2fac 100644
--- a/components/cronet/android/cronet_url_request.cc
+++ b/components/cronet/android/cronet_url_request.cc
@@ -285,6 +285,8 @@ static jstring GetHttpStatusText(JNIEnv* env,
   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();
 }