Cronet bots crashing all the time |
|||
Issue descriptionLooks like every test is crashing: https://build.chromium.org/p/chromium.android/console Call stack is something like this: StringObject::utfLength() const+17 base::android::ConvertJavaStringToUTF8(_JNIEnv*, _jstring*, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >*)+16 base::android::ConvertJavaStringToUTF8(_JNIEnv*, base::android::JavaRef<_jstring*> const&)+20 Java_org_chromium_net_CronetUrlRequestContext_nativeCreateRequestContextConfig+106 Java_org_chromium_net_CronetUrlRequestContext_nativeCreateRequestContextConfig This looks like the most recent successful run: https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Data%20Reduction%20Proxy%20Builder/builds/1618 which used r383999 This looks like the first failed run: https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20ARMv6%20Builder/builds/1586 which used r384013 This looks like the culprit: https://codereview.chromium.org/1828193002 I think we should revert ASAP and investigate a fix later.
,
Mar 31 2016
Seems like this should be on trybots if it can be broken by changes to general android code. :/
,
Mar 31 2016
We have one trybot called android_cronet_tester.
,
Mar 31 2016
Issue 599506 has been merged into this issue.
,
Mar 31 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1d9c5cae125a244f561276d2e8824dd2ec04dcdc commit 1d9c5cae125a244f561276d2e8824dd2ec04dcdc Author: bauerb <bauerb@chromium.org> Date: Thu Mar 31 19:10:47 2016 CroNet: Allow null strings in config when creating a UrlRequestContextConfig. BUG= 599331 Review URL: https://codereview.chromium.org/1850573003 Cr-Commit-Position: refs/heads/master@{#384355} [modify] https://crrev.com/1d9c5cae125a244f561276d2e8824dd2ec04dcdc/components/cronet/android/cronet_url_request_context_adapter.cc
,
Apr 5 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by bauerb@chromium.org
, Mar 31 2016