[Cronet] Crash in SdchTest#testSdchEnabled |
|||
Issue descriptionhttps://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Lollipop%20Builder/builds/3851/steps/cronet_test_instrumentation_apk/logs/stdio 4b057: 11-10 15:07:43.579 13999 14030 W System.err: java.lang.NullPointerException: Attempt to invoke virtual method 'void android.os.ConditionVariable.open()' on a null object reference 4b057: 11-10 15:07:43.579 13999 14030 W System.err: at org.chromium.net.SdchTest$DictionaryAddedObserver.onDictionaryAdded(SdchTest.java:158)
,
Nov 10 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9049bdbc90b226e77758b7ae845584422f0663ac commit 9049bdbc90b226e77758b7ae845584422f0663ac Author: xunjieli <xunjieli@chromium.org> Date: Thu Nov 10 18:58:47 2016 [Cronet] Fix SdchTest.DictionaryAddedObserver The test run linked by the bug below seems to suggest that mBlock is null when we call mBlock.open(). This CL changes initialization of mBlock to be more compliant with the standard way of initialization member variables. BUG= 664176 Review-Url: https://codereview.chromium.org/2489303003 Cr-Commit-Position: refs/heads/master@{#431306} [modify] https://crrev.com/9049bdbc90b226e77758b7ae845584422f0663ac/components/cronet/android/test/javatests/src/org/chromium/net/SdchTest.java
,
Nov 10 2016
Leaving this open for a bit since we don't really know what the cause is. I am interested to see if we get more failed runs.
,
Dec 16 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by xunji...@chromium.org
, Nov 10 2016