[Cronet] Disable testMockReadDataAsyncError as it breaks with WeakPtr changes.

components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestTest.java

 // Copyright 2014 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;
 
 import android.os.Build;
 import android.os.ConditionVariable;
 import android.os.StrictMode;
 import android.support.test.filters.SmallTest;
 
 import org.chromium.base.Log;
+import org.chromium.base.test.util.DisabledTest;
 import org.chromium.base.test.util.Feature;
 import org.chromium.net.TestUrlRequestCallback.FailureType;
 import org.chromium.net.TestUrlRequestCallback.ResponseStep;
 import org.chromium.net.impl.CronetUrlRequest;
 import org.chromium.net.impl.UrlResponseInfoImpl;
 import org.chromium.net.test.FailurePhase;
 
 import java.io.IOException;
 import java.net.ConnectException;
 import java.nio.ByteBuffer;
@@ skipping 619 matching lines @@
         assertEquals(200, callback.mResponseInfo.getHttpStatusCode());
         assertEquals(15, callback.mResponseInfo.getReceivedByteCount());
         assertNotNull(callback.mError);
         assertEquals(arbitraryNetError,
                 ((NetworkException) callback.mError).getCronetInternalErrorCode());
         assertEquals(0, callback.mRedirectCount);
         assertTrue(callback.mOnErrorCalled);
         assertEquals(ResponseStep.ON_FAILED, callback.mResponseStep);
     }
 
+    @DisabledTest(message = "crbug.com/738183")
     @SmallTest
     @Feature({"Cronet"})
     @OnlyRunNativeCronet // Java impl doesn't support MockUrlRequestJobFactory
     public void testMockReadDataAsyncError() throws Exception {
         final int arbitraryNetError = -5;
         TestUrlRequestCallback callback =
                 startAndWaitForComplete(MockUrlRequestJobFactory.getMockUrlWithFailure(
                         FailurePhase.READ_ASYNC, arbitraryNetError));
         assertEquals(200, callback.mResponseInfo.getHttpStatusCode());
         assertEquals(15, callback.mResponseInfo.getReceivedByteCount());
@@ skipping 1485 matching lines @@
             // The other quirk of this is that from teardown(), JavaCronetEngine.shutdown() is
             // called which calls ExecutorService.shutdown() which doesn't wait for the thread to
             // finish running tasks, and then teardown() calls GC looking for leaks. One possible
             // modification would be to expose the ExecutorService and then have tests call
             // awaitTermination() but this would complicate things, and adding a 1s sleep() to
             // allow the ExecutorService to terminate did not increase the chances of catching the
             // leak.
         }
     }
 }