Add SetSupportsQuic method to explicitly specify server that supports QUIC.

components/cronet/android/chromium_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/chromium_url_request.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/macros.h"
 #include "components/cronet/android/url_request_adapter.h"
@@ skipping 369 matching lines @@
   }
 
   // Some implementations (notably HttpURLConnection) include a mapping for the
   // null key; in HTTP's case, this maps to the HTTP status line.
   ScopedJavaLocalRef<jstring> status_line =
       ConvertUTF8ToJavaString(env, headers->GetStatusLine());
   Java_ChromiumUrlRequest_onAppendResponseHeader(
       env, object, headersMap, NULL, status_line.Release());
 }
 
+static jstring GetNegotiatedProtocol(JNIEnv* env,
+                                     jobject object,
+                                     jlong urlRequestAdapter) {
+  URLRequestAdapter* request =
+      reinterpret_cast<URLRequestAdapter*>(urlRequestAdapter);
+  if (request == NULL) {

[mmenke, 2014/09/09 14:53:35]

Looks like we're pretty inconsistent about checking this.  (See, for example,
AppendChunk, GetErrorString, and GetHttpStatusCode).  If it can't happen, we
shouldn't be checking for it.  If it can, we should be more consistent (And have
tests).

[mef, 2014/09/10 16:48:17]

Acknowledged. I think we should have these checks everywhere, but I'm not sure
whether that belongs to this CL.

+    return NULL;
+  }
+  std::string negotiated_protocol = request->GetNegotiatedProtocol();

[mmenke, 2014/09/09 14:53:35]

If we haven't negotiated a protocol yet, I assume this is an empty string?  If
so, think we should also be returning that in the request==NULL case, if that
code is worth having (I'm skeptical that it is, unless it can actually happen -
and if it can, we should probably be throwing exceptions, actually).

[mef, 2014/09/10 16:48:17]

Done.

+  return ConvertUTF8ToJavaString(env, negotiated_protocol.c_str()).Release();
+}
+
 }  // namespace cronet