Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(957)

Unified Diff: components/cronet/android/test/src/org/chromium/net/CronetTestUtil.java

Issue 2339223002: Cronet API Refactoring (Closed)
Patch Set: Rebased onto Charles change + Paul's Comments Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/cronet/android/test/src/org/chromium/net/CronetTestUtil.java
diff --git a/components/cronet/android/test/src/org/chromium/net/CronetTestUtil.java b/components/cronet/android/test/src/org/chromium/net/CronetTestUtil.java
index c9ed886763002b02f0e9940da48e8f7125c56048..ffbc56ae71ee8e21b4a961b53d3d12c88a119ef4 100644
--- a/components/cronet/android/test/src/org/chromium/net/CronetTestUtil.java
+++ b/components/cronet/android/test/src/org/chromium/net/CronetTestUtil.java
@@ -7,6 +7,7 @@ package org.chromium.net;
import android.os.ConditionVariable;
import org.chromium.base.annotations.JNINamespace;
+import org.chromium.net.impl.CronetEngineBuilderImpl;
import org.chromium.net.impl.CronetUrlRequest;
import org.json.JSONException;
import org.json.JSONObject;
@@ -65,5 +66,18 @@ public class CronetTestUtil {
return nativeGetLoadFlags(((CronetUrlRequest) urlRequest).getUrlRequestAdapterForTesting());
}
+ public static void setMockCertVerifierForTesting(
+ CronetEngine.Builder builder, long mockCertVerifier) {
+ getCronetEngineBuilderImpl(builder).setMockCertVerifierForTesting(mockCertVerifier);
+ }
+
+ public static void setLibraryName(CronetEngine.Builder builder, String libName) {
+ getCronetEngineBuilderImpl(builder).setLibraryName(libName);
+ }
+
+ public static CronetEngineBuilderImpl getCronetEngineBuilderImpl(CronetEngine.Builder builder) {
pauljensen 2016/09/26 14:51:22 is this used outside this class? if not, can we m
kapishnikov 2016/09/27 18:38:26 Yes, it is used in CronetUrlRequestContextTest.
+ return (CronetEngineBuilderImpl) builder.getBuilderDelegate();
+ }
+
private static native int nativeGetLoadFlags(long urlRequest);
}

Powered by Google App Engine
This is Rietveld 408576698