Cronet API Refactoring

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

+// Copyright 2016 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;
+
+/**
+ * {@link UrlRequest} that exposes experimental feature.
+ * {@hide} since this class exposes experimental features that should be hidden.
+ */
+public abstract class ExperimentalUrlRequest extends UrlRequest {
+    /**
+     * Builder for building {@link UrlRequest}.
+     */
+    public abstract static class Builder extends UrlRequest.Builder {
+        /**
+         * Disables connection migration for the request if enabled for
+         * the session.
+         * @return the builder to facilitate chaining.
+         */
+        public abstract Builder disableConnectionMigration();
+
+        /**
+         * Associates the annotation object with this request. May add more than one.
+         * Passed through to a {@link RequestFinishedInfo.Listener},
+         * see {@link RequestFinishedInfo#getAnnotations}.
+         *
+         * @param annotation an object to pass on to the {@link RequestFinishedInfo.Listener} with a
+         * {@link RequestFinishedInfo}.
+         * @return the builder to facilitate chaining.
+         */
+        public abstract Builder addRequestAnnotation(Object annotation);

[pauljensen, 2016/09/26 14:51:21]

how come this class doesn't have a build() method, like
ExperimentalCronetEngine.Builder?

[kapishnikov, 2016/09/27 18:38:25]

build() is inherited from UrlRequest.Builder. The actual implementation of the
method is in UrlRequestBuilderImpl.

[pauljensen, 2016/09/27 19:08:09]

Agreed to add it and make it abstract.

[kapishnikov, 2016/09/28 00:20:49]

Done. Same for ExperimentalBidirectionalStream.Builder 

+    }
+}