Added Network Quality Estimator Real-time interface to Cronet

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

Index: components/cronet/android/java/src/org/chromium/net/UrlRequestContext.java
diff --git a/components/cronet/android/java/src/org/chromium/net/UrlRequestContext.java b/components/cronet/android/java/src/org/chromium/net/UrlRequestContext.java
index cc0e63da4ec04af087ea1b8412088573b282c8a1..f881241e6a2559e8fdb567ab94c46b5b6a816989 100644
--- a/components/cronet/android/java/src/org/chromium/net/UrlRequestContext.java
+++ b/components/cronet/android/java/src/org/chromium/net/UrlRequestContext.java
@@ -94,6 +94,40 @@ public abstract class UrlRequestContext {
     public abstract void stopNetLog();
 
     /**
+     * Enables the network quality estimator. This must be called before RTT

[pauljensen, 2015/09/01 10:59:10]

You abbreviate the first occurrence of RTT but then write out "round trip time"
later.  Please write it out the first time.

[bengr, 2015/09/01 23:07:39]

Done.

+     * and throughput observers are added. Set both parameters to false
+     * for default behavior.
+     * @param useLocalHostRequests Include requests to localhost in estimates.
+     * @param useSmallerResponses Include small responses in throughput estimates.

[pauljensen, 2015/09/01 10:59:10]

AFAIK these parameters are only for testing.  Perhaps we should have a separate
API to enable for testing; it could be either package-private or @hide.

[bengr, 2015/09/01 23:07:39]

Done.

+     */
+    public abstract void enableNetworkQualityEstimator(
+            boolean useLocalHostRequests, boolean useSmallerResponses);
+
+    /**
+     * Adds an observer of round trip time observations.

[pauljensen, 2015/09/01 10:59:10]

Please make this more verbose.  When and under what circumstances does this get
called?  Ditto for the other functions.

[bengr, 2015/09/01 23:07:39]

Done.

+     * @param observer The observer of round trip times.
+     */
+    public abstract void addRTTObserver(NetworkQualityRTTObserver observer);
+
+    /**
+     * Removes an observer of round trip times if on the observer list.
+     * @param observer The observer of round trip times.
+     */
+    public abstract void removeRTTObserver(NetworkQualityRTTObserver observer);
+
+    /**
+     * Adds an observer of throughout.
+     * @param observer The observer of throughput.
+     */
+    public abstract void addThroughputObserver(NetworkQualityThroughputObserver observer);
+
+    /**
+     * Removes an observer of throughput.
+     * @param observer The observer of throughput.
+     */
+    public abstract void removeThroughputObserver(NetworkQualityThroughputObserver observer);
+
+    /**
      * Creates a {@link UrlRequestContext} with the given
      * {@link UrlRequestContextConfig}.
      * @param context Android {@link Context}.