Add SetSupportsQuic method to explicitly specify server that supports QUIC.

components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequestContext.java

 // 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.
 
 package org.chromium.net;
 
 import android.content.Context;
 import android.os.ConditionVariable;
 import android.os.Process;
 import android.util.Log;
@@ skipping 68 matching lines @@
     }
 
     /**
      * Stops NetLog logging and flushes file to disk. If a logging session is
      * not in progress this call is ignored.
      */
     public void stopNetLog() {
         nativeStopNetLog(mChromiumUrlRequestContextAdapter);
     }
 
+    /**
+     * Explicitly mark |server| as supporting QUIC with given |probability|.
+     * Note that DISK CACHE must be enabled to take advantage or 0RTT QUIC.
+     *
+     * @param server URL of the server that supports QUIC
+     * @param alternatePort to use for QUIC
+     * @param probability that server supports QUIC (0..1)

[mmenke, 2014/09/05 17:20:34]

This seems like a bad idea to me.

[mef, 2014/09/05 18:15:42]

Done.

+     */
+    public void setSupportsQuic(String server,

[mmenke, 2014/09/05 17:20:33]

Suggest calling this setQuicHint, which leaves a bit more room for non-QUIC
fallback.

[mmenke, 2014/09/05 17:20:34]

I strongly suggest we throw an exception if |server| is not a URL.  Think it
would be easy to accidentally do "setSupportsQuic("www.google.com", ...)". 
Could make nativeSetSupportsQuic return false on invalid URLs (And maybe
non-http/https ones as well), and throw on that - think that's simpler than
validating in Java.

[mef, 2014/09/05 18:15:42]

Done.

[mef, 2014/09/05 18:15:42]

Done.

[Charles, 2014/09/05 18:16:22]

This isn't very useful, as API goes - a normal user has no access to the
ChromiumUrlRequestContext object. You should put this configuration in the
HttpUrlRequestFactoryConfig.

[mef, 2014/09/05 19:07:28]

Acknowledged. This is geared towards advanced user. :) ChromiumUrlRequestContext
is accessible using ChromiumUrlRequestFactory.getRequestContext() method.

I have no objections to putting this into HttpUrlRequestFactoryConfig, and will
happily do so if it makes sense.

Beauty of this method is that it could actually be called after instantiation of
factory/context.
 

+                                int alternatePort,
+                                float probability) {
+        nativeSetSupportsQuic(mChromiumUrlRequestContextAdapter, server,
+                alternatePort, probability);
+    }
+
     @CalledByNative
     private void initNetworkThread() {
         Thread.currentThread().setName("ChromiumNet");
         Process.setThreadPriority(Process.THREAD_PRIORITY_BACKGROUND);
         mStarted.open();
     }
 
     @Override
     protected void finalize() throws Throwable {
         nativeReleaseRequestContextAdapter(mChromiumUrlRequestContextAdapter);
@@ skipping 19 matching lines @@
         }
         return loggingLevel;
     }
 
     // Returns an instance ChromiumUrlRequestContextAdapter to be stored in
     // mChromiumUrlRequestContextAdapter.
     private native long nativeCreateRequestContextAdapter(Context context,
             String userAgent, int loggingLevel, String config);
 
     private native void nativeReleaseRequestContextAdapter(
-            long ChromiumUrlRequestContextAdapter);
+            long chromiumUrlRequestContextAdapter);
 
     private native void nativeInitializeStatistics();
 
     private native String nativeGetStatisticsJSON(String filter);
 
     private native void nativeStartNetLogToFile(
-            long ChromiumUrlRequestContextAdapter, String fileName);
+            long chromiumUrlRequestContextAdapter, String fileName);
 
-    private native void nativeStopNetLog(long ChromiumUrlRequestContextAdapter);
+    private native void nativeStopNetLog(long chromiumUrlRequestContextAdapter);
+
+    private native void nativeSetSupportsQuic(
+            long chromiumUrlRequestContextAdapter, String server,
+            int alternatePort, float probability);
 }