Index: components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java |
diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java b/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java |
index ce07d6bdca5eb22145ced74721f3991f8d7bd926..45c76306d0d3498e55156c04d8b7c05346c1e86f 100644 |
--- a/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java |
+++ b/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java |
@@ -960,4 +960,35 @@ public class CronetUrlRequestContextTest extends CronetTestBase { |
// Verifies that CronetEngine.Builder config from testCronetEngineBuilderConfig() is properly |
// translated to a native UrlRequestContextConfig. |
private static native void nativeVerifyUrlRequestContextConfig(long config, String storagePath); |
+ |
+ private static class TestBadLibraryLoader extends CronetEngine.Builder.LibraryLoader { |
+ private int mWasCalled = 0; |
xunjieli
2016/01/25 18:24:09
suggest using a boolean.
mgersh
2016/01/25 19:37:31
Done.
|
+ |
+ public void loadLibrary(String libName) { |
+ // Report that this method was called, but don't load the library |
+ mWasCalled = 1; |
+ } |
+ |
+ int wasCalled() { |
+ return mWasCalled; |
+ } |
+ } |
+ |
+ @SmallTest |
+ @Feature({"Cronet"}) |
+ public void testSkipLibraryLoading() throws Exception { |
+ CronetEngine.Builder builder = new CronetEngine.Builder(getContext()); |
+ TestBadLibraryLoader loader = new TestBadLibraryLoader(); |
+ builder.setLibraryLoader(loader).setLibraryName("cronet_tests"); |
+ try { |
+ // ensureInitialized() calls native code to check the version right after library load |
+ // and will error with the message below if library loading was skipped |
+ CronetLibraryLoader.ensureInitialized(getContext(), builder); |
+ fail("Native library should not be loaded"); |
+ } catch (UnsatisfiedLinkError e) { |
+ assertTrue(e.getMessage().contains( |
+ "No implementation found for java.lang.String org.chromium.net")); |
+ assertEquals(loader.wasCalled(), 1); |
+ } |
+ } |
} |