Delete old NetworkQualityEstimator Cronet API

components/cronet/android/api/src/org/chromium/net/CronetEngine.java

 // Copyright 2015 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.annotation.SuppressLint;
 import android.content.Context;
 import android.net.http.HttpResponseCache;
 import android.support.annotation.IntDef;
@@ skipping 941 matching lines @@
      * useful data as no metrics have yet been collected.
      *
      * @return differences in metrics collected by Cronet, since the last call
      *         to {@code getGlobalMetricsDeltas()}, serialized as a
      *         <a href=https://developers.google.com/protocol-buffers>protobuf
      *         </a>.
      */
     public abstract byte[] getGlobalMetricsDeltas();
 
     /**
-     * Sets the executor which will be used to notify RequestFinished
-     *             listeners, and to notify network quality RTT listeners
-     *             that do not provide an executor.
-     * TODO(tbansal):  http://crbug.com/618034 Remove this API. In short term,
-     * once all Cronet embedders supply a valid executor with
-     * NetworkQualityRTTListener, update the above comment to reflect that
-     * {@link executor} is only used to notify RequestFinishedListeners.
-     * @hide as it's a prototype.
-     */
-    public abstract void setRequestFinishedListenerExecutor(Executor executor);
-
-    /**
-     * Enables the network quality estimator, which collects and reports
-     * measurements of round trip time (RTT) and downstream throughput at
-     * various layers of the network stack. After enabling the estimator,
-     * listeners of RTT and throughput can be added with
-     * {@link #addRttListener} and {@link #addThroughputListener} and
-     * removed with {@link #removeRttListener} and
-     * {@link #removeThroughputListener}. The estimator uses memory and CPU
-     * only when enabled.
-     * @param executor an executor that will be used to notified all
-     *            added RTT and throughput listeners.
-     * TODO(tbansal):  http://crbug.com/618034 Remove this API.
-     * @hide as it's a prototype.
-     */
-    public abstract void enableNetworkQualityEstimator(Executor executor);
-
-    /**
      * Configures the network quality estimator for testing. This must be called
      * before round trip time and throughput listeners are added, and after the
      * network quality estimator has been enabled.
      * @param useLocalHostRequests include requests to localhost in estimates.
      * @param useSmallerResponses include small responses in throughput
      * estimates.
      * @hide as it's a prototype.
      */
     public abstract void configureNetworkQualityEstimatorForTesting(
             boolean useLocalHostRequests, boolean useSmallerResponses);
@@ skipping 129 matching lines @@
      * Registers a listener that gets called after the end of each request with the request info.
      *
      * <p>This must be called after {@link #enableNetworkQualityEstimator} and will throw an
      * exception otherwise.
      *
      * <p>The listener is called on the {@link java.util.concurrent.Executor} that
      * is passed to {@link #enableNetworkQualityEstimator}.
      *
      * @param listener the listener for finished requests.
      *
-     * TODO(tbansal):  http://crbug.com/618034 Remove this API, once all embedders have switched to
-     * using a request finished listener that provides its own executor.
      * @hide as it's a prototype.
      */
     public abstract void addRequestFinishedListener(RequestFinishedListener listener);
 
     /**
      * Removes a finished request listener.
      *
      * @param listener the listener to remove.
      *
-     * TODO(tbansal):  http://crbug.com/618034 Remove this API, once all embedders have switched to
-     * using a request finished listener that provides its own executor.
      * @hide it's a prototype.
      */
     public abstract void removeRequestFinishedListener(RequestFinishedListener listener);
 
     /**
      * Information about a finished request. Passed to {@link RequestFinishedListener}.
      *
      * @hide as it's a prototype.
      */
     public static final class UrlRequestInfo {
@@ skipping 100 matching lines @@
 
         /**
          * Returns total bytes received over the network transport layer, or null if not collected.
          */
         @Nullable
         public Long getReceivedBytesCount() {
             return mReceivedBytesCount;
         }
     }
 }