Chromium Code Reviews| 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); |
| + } |
| + } |
| } |