refactor(common): move ReproducibleHttpRequestDuplicator into common, package-private

The duplicator was public only to be reachable across the internal.common ->
common package boundary from ReproducibleHttpRequest#toDuplicator. It has no
external consumers, so co-locating it with its sole caller in
com.linecorp.armeria.common lets it (and its constructor) become package-private,
matching the sibling DefaultHttpRequestDuplicator. No behavior change.

Co-authored-by: Isaac

Author: yzfeng2020

core/src/main/java/com/linecorp/armeria/common/ReproducibleHttpRequest.java

@@ -24,7 +24,6 @@
 import org.reactivestreams.Publisher;
 
 import com.linecorp.armeria.common.stream.StreamMessage;
-import com.linecorp.armeria.internal.common.ReproducibleHttpRequestDuplicator;
 import com.linecorp.armeria.internal.common.stream.NonOverridableStreamMessageWrapper;
 
 import io.netty.util.concurrent.EventExecutor;

…n/ReproducibleHttpRequestDuplicator.java …n/ReproducibleHttpRequestDuplicator.javacore/src/main/java/com/linecorp/armeria/internal/common/ReproducibleHttpRequestDuplicator.java renamed to core/src/main/java/com/linecorp/armeria/common/ReproducibleHttpRequestDuplicator.java core/src/main/java/com/linecorp/armeria/internal/common/ReproducibleHttpRequestDuplicator.java renamed to core/src/main/java/com/linecorp/armeria/common/ReproducibleHttpRequestDuplicator.java

@@ -14,7 +14,7 @@
  * under the License.
  */
 
-package com.linecorp.armeria.internal.common;
+package com.linecorp.armeria.common;
 
 import static java.util.Objects.requireNonNull;
 
@@ -25,10 +25,6 @@
 import java.util.Set;
 import java.util.function.Supplier;
 
-import com.linecorp.armeria.common.HttpObject;
-import com.linecorp.armeria.common.HttpRequest;
-import com.linecorp.armeria.common.HttpRequestDuplicator;
-import com.linecorp.armeria.common.RequestHeaders;
 import com.linecorp.armeria.common.annotation.Nullable;
 import com.linecorp.armeria.common.stream.StreamMessage;
 
@@ -53,7 +49,7 @@
  * {@code duplicate}, and {@code duplicate} then throws instead of returning a request that would never
  * be torn down.
  */
-public final class ReproducibleHttpRequestDuplicator implements HttpRequestDuplicator {
+final class ReproducibleHttpRequestDuplicator implements HttpRequestDuplicator {
 
     private final RequestHeaders headers;
     private final Supplier<? extends StreamMessage<? extends HttpObject>> bodyFactory;
@@ -65,7 +61,7 @@ public final class ReproducibleHttpRequestDuplicator implements HttpRequestDupli
     @Nullable
     private Throwable abortCause;
 
-    public ReproducibleHttpRequestDuplicator(
+    ReproducibleHttpRequestDuplicator(
             RequestHeaders headers,
             Supplier<? extends StreamMessage<? extends HttpObject>> bodyFactory) {
         this.headers = requireNonNull(headers, "headers");

…producibleHttpRequestDuplicatorTest.java …producibleHttpRequestDuplicatorTest.javacore/src/test/java/com/linecorp/armeria/internal/common/ReproducibleHttpRequestDuplicatorTest.java renamed to core/src/test/java/com/linecorp/armeria/common/ReproducibleHttpRequestDuplicatorTest.java core/src/test/java/com/linecorp/armeria/internal/common/ReproducibleHttpRequestDuplicatorTest.java renamed to core/src/test/java/com/linecorp/armeria/common/ReproducibleHttpRequestDuplicatorTest.java

@@ -14,7 +14,7 @@
  * under the License.
  */
 
-package com.linecorp.armeria.internal.common;
+package com.linecorp.armeria.common;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
@@ -24,11 +24,6 @@
 
 import org.junit.jupiter.api.Test;
 
-import com.linecorp.armeria.common.HttpData;
-import com.linecorp.armeria.common.HttpMethod;
-import com.linecorp.armeria.common.HttpObject;
-import com.linecorp.armeria.common.HttpRequest;
-import com.linecorp.armeria.common.RequestHeaders;
 import com.linecorp.armeria.common.stream.StreamMessage;
 
 class ReproducibleHttpRequestDuplicatorTest {