Expose Cronet Url requests with priorities.

components/cronet/android/java/src/org/chromium/net/UrlRequestContext.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.util.Log;
 
 import java.lang.reflect.Constructor;
 import java.util.concurrent.Executor;
 
 /**
  * A context for {@link UrlRequest}'s, which uses the best HTTP stack
  * available on the current platform.
  */
 public abstract class UrlRequestContext {
     private static final String TAG = "UrlRequestFactory";
     private static final String CRONET_URL_REQUEST_CONTEXT =
             "org.chromium.net.CronetUrlRequestContext";
 
     /**
-     * Creates an UrlRequest object. All UrlRequest functions must
-     * be called on the Executor's thread, and all callbacks will be called
-     * on the Executor's thread as well. Executor must not run tasks on the
+     * Creates a {@link org.chromium.net.UrlRequest} object. All callbacks will
+     * be called on the Executor's thread. Executor must not run tasks on the
      * current thread to prevent network jank and exception during shutdown.
      *
-     * createRequest itself may be called on any thread.
      * @param url URL for the request.
      * @param listener Callback interface that gets called on different events.
      * @param executor Executor on which all callbacks will be called.
      * @return new request.
      */
     public abstract UrlRequest createRequest(String url,
             UrlRequestListener listener, Executor executor);
 
     /**
+     * Creates a {@link org.chromium.net.UrlRequest} object. All callbacks will

[xunjieli, 2015/06/05 15:29:50]

nit: I think if we are refering to the class in the same package, the prefix can
be omitted, so this can be "{@link UrlRequest}". (x2)

[mkirsche, 2015/06/05 15:43:55]

Done.

+     * be called on the Executor's thread. Executor must not run tasks on the
+     * current thread to prevent network jank and exception during shutdown.
+     *
+     * @param url URL for the request.
+     * @param listener Callback interface that gets called on different events.
+     * @param executor Executor on which all callbacks will be called.
+     * @param priority Priority of the request which should be one of the
+                  REQUEST_PRIORITY_* values in UrlRequest.java

[xunjieli, 2015/06/05 15:29:50]

nit: Use "{@link UrlRequest}"

[mkirsche, 2015/06/05 15:43:55]

Done.

+     * @return new request.
+     */
+    public abstract UrlRequest createRequest(String url,
+            UrlRequestListener listener, Executor executor, int priority);
+
+    /**
      * @return true if the context is enabled.
      */
     public abstract boolean isEnabled();
 
     /**
      * @return a human-readable version string of the context.
      */
     public abstract String getVersionString();
 
     /**
@@ skipping 72 matching lines @@
         } catch (ClassNotFoundException e) {
             // Leave as null.
         } catch (Exception e) {
             throw new IllegalStateException(
                     "Cannot instantiate: " + CRONET_URL_REQUEST_CONTEXT,
                     e);
         }
         return urlRequestContext;
     }
 }