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;
@@ skipping 17 matching lines @@
      * 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 an UrlRequest object. All UrlRequest functions must

[xunjieli, 2015/06/05 15:02:51]

nit: a UrlRequest. Please also surround it with {@link}.

[mkirsche, 2015/06/05 15:12:21]

Done.

+     * be called on the Executor's thread, and all callbacks will be called

[xunjieli, 2015/06/05 15:02:51]

Since you are here, could you remove "All UrlRequest functions must be called on
the Executor's thread, and"? That part is misleading. Please also edit the
javadoc for the other method.

[mkirsche, 2015/06/05 15:12:21]

Done.

+     * on the Executor's thread as well. 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.

[xunjieli, 2015/06/05 15:02:51]

Could you remove "createRequest itself may be called on any thread."? That's
redundant once we remove the part above.

[mkirsche, 2015/06/05 15:12:21]

Done.

+     * @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 - should be one of the

[xunjieli, 2015/06/05 15:02:51]

Why is there a dash here?

[mkirsche, 2015/06/05 15:12:21]

Done.

+                  REQUEST_PRIORITY_* values in UrlRequest.java
+     * @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;
     }
 }