New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 728554 link

Starred by 2 users

Issue metadata

Status: Started
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

WebRtcTwoDeviceDepthCaptureBrowserTest's flaky in asan-clang-phone

Project Member Reported by perezju@chromium.org, Jun 1 2017

Issue description

The following tests have recently started showing flaky crashes on asan-clang-phone:

test                                                                                        build                     status
WebRtcTwoDeviceDepthCaptureBrowserTest.GetBothStreamsAndCheckForFeaturesPresence (CRASHED)   2850  --F-~F~-~-------~~--~---~
WebRtcTwoDeviceDepthCaptureBrowserTest.GetStreamsByVideoKind (CRASHED)                       2850  -F-F~-~-~-------~~--~---~

Build: https://uberchromegw.corp.google.com/i/internal.client.clank/builders/asan-clang-phone/builds/2849
Log: https://luci-logdog.appspot.com/v/?s=chrome%2Fbb%2Finternal.client.clank%2Fasan-clang-phone%2F2849%2F%2B%2Frecipes%2Fsteps%2Fcontent_browsertests%2F0%2Fstdout

Build: https://uberchromegw.corp.google.com/i/internal.client.clank/builders/asan-clang-phone/builds/2848
Log: https://luci-logdog.appspot.com/v/?s=chrome%2Fbb%2Finternal.client.clank%2Fasan-clang-phone%2F2848%2F%2B%2Frecipes%2Fsteps%2Fcontent_browsertests%2F0%2Fstdout

From one of the logs:

C 6820.021s Main  Detailed Logs
C 6820.021s Main  ********************************************************************************
C 6820.025s Main  [CRASH] WebRtcTwoDeviceDepthCaptureBrowserTest.GetStreamsByVideoKind:
C 6820.025s Main  [ RUN      ] WebRtcTwoDeviceDepthCaptureBrowserTest.GetStreamsByVideoKind
C 6820.025s Main  [WARNING:dns_config_service_posix.cc(326)] Failed to read DnsConfig.
C 6820.025s Main  [ERROR:devtools_http_handler.cc(226)] Cannot start http server for devtools. Stop devtools.
C 6820.025s Main  [ERROR:child_process_launcher_helper_posix.cc(118)] Ignoring invalid file assets/snapshot_blob_64.bin
C 6820.025s Main  [WARNING:simple_synchronous_entry.cc(1102)] Could not open platform files for entry.
C 6820.025s Main  [INFO:CONSOLE(140)] "Calling getStreamsByVideoKind", source: http://127.0.0.1:43998/media/getusermedia-depth-capture.html (140)
C 6820.025s Main  [INFO:SkFontMgr_android_parser.cpp(609)] [SkFontMgr Android Parser] '/system/etc/fonts.xml' could not be opened
C 6820.025s Main  
C 6820.025s Main  [INFO:CONSOLE(29)] "Test Success", source: http://127.0.0.1:43998/media/webrtc_test_utilities.js (29)
C 6820.025s Main  [INFO:SkFontMgr_android_parser.cpp(609)] [SkFontMgr Android Parser] '/vendor/etc/fallback_fonts.xml' could not be opened
C 6820.025s Main  
C 6820.025s Main  [FATAL:interface_endpoint_client.cc(33)] Check failed: !is_valid. The callback passed to ImageCapture::GetCapabilities() was never run.
C 6820.025s Main  #00 0x7a777501 /data/app-lib/org.chromium.content_browsertests_apk-1/libbase.cr.so+0x00173501
C 6820.025s Main  #01 0x7bbaf515 /data/app-lib/org.chromium.content_browsertests_apk-1/libbindings.cr.so+0x00040515
C 6820.025s Main  #02 0x7bbaebfd /data/app-lib/org.chromium.content_browsertests_apk-1/libbindings.cr.so+0x0003fbfd
C 6820.025s Main  #03 0x909158eb /data/app-lib/org.chromium.content_browsertests_apk-1/libcontent.cr.so+0x0230b8eb
[...]
 

Comment 1 by awdf@chromium.org, Jun 6 2017

Cc: aleksand...@intel.com chfremer@chromium.org
The first of those just failed again on asan-clang-phone:

There's a fuller stack available at the stack_tool_with_logcat_dump step (grep for ImageCapture::GetCapabilities(), pasting here in case it is helpful:

signal 6 (SIGABRT) at 0x00007260 (code=-6), thread 29305 (CrRendererMain)
[FATAL:interface_endpoint_client.cc(33)] Check failed: !is_valid. The callback passed to ImageCapture::GetCapabilities() was never run.
Stack Trace:
  RELADDR   FUNCTION                                                                                                                                                                                                                                                                                                                                                                                                               FILE:LINE
  00173469  logging::LogMessage::~LogMessage()+504                                                                                                                                                                                                                                                                                                                                                                                 /b/build/slave/asan-clang-phone/build/src/base/logging.cc:552
  00040515  mojo::(anonymous namespace)::DCheckIfInvalid(base::WeakPtr<mojo::InterfaceEndpointClient> const&, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > const&)+216                                                                                                                                                                                                           /b/build/slave/asan-clang-phone/build/src/mojo/public/cpp/bindings/lib/interface_endpoint_client.cc:33
  0003fbfd  mojo::(anonymous namespace)::ResponderThunk::DCheckInvalid(std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > const&)+180                                                                                                                                                                                                                                                  /b/build/slave/asan-clang-phone/build/src/mojo/public/cpp/bindings/lib/interface_endpoint_client.cc:93
  0235555b  media::mojom::ImageCapture_GetCapabilities_ProxyToResponder::~ImageCapture_GetCapabilities_ProxyToResponder()+210                                                                                                                                                                                                                                                                                                      /b/build/slave/asan-clang-phone/build/src/out/Debug/gen/media/capture/mojo/image_capture.mojom.cc:543
  v------>  operator()                                                                                                                                                                                                                                                                                                                                                                                                             /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/memory:2431
  v------>  reset                                                                                                                                                                                                                                                                                                                                                                                                                  /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/memory:2630
  v------>  ~unique_ptr                                                                                                                                                                                                                                                                                                                                                                                                            /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/memory:2598
  02354c23  base::internal::PassedWrapper<std::__ndk1::unique_ptr<media::mojom::ImageCapture_GetCapabilities_ProxyToResponder, std::__ndk1::default_delete<media::mojom::ImageCapture_GetCapabilities_ProxyToResponder> > >::~PassedWrapper()+28                                                                                                                                                                                   /b/build/slave/asan-clang-phone/build/src/base/bind_helpers.h:267
  v------>  ~__tuple_leaf                                                                                                                                                                                                                                                                                                                                                                                                          /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/tuple:180
  v------>  ~tuple                                                                                                                                                                                                                                                                                                                                                                                                                 /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/tuple:486
  0235547d  base::internal::BindState<void (media::mojom::ImageCapture_GetCapabilities_ProxyToResponder::*)(mojo::StructPtr<media::mojom::PhotoCapabilities>), base::internal::PassedWrapper<std::__ndk1::unique_ptr<media::mojom::ImageCapture_GetCapabilities_ProxyToResponder, std::__ndk1::default_delete<media::mojom::ImageCapture_GetCapabilities_ProxyToResponder> > > >::~BindState()+12                                  /b/build/slave/asan-clang-phone/build/src/base/bind_internal.h:482
  02355465  base::internal::BindState<void (media::mojom::ImageCapture_GetCapabilities_ProxyToResponder::*)(mojo::StructPtr<media::mojom::PhotoCapabilities>), base::internal::PassedWrapper<std::__ndk1::unique_ptr<media::mojom::ImageCapture_GetCapabilities_ProxyToResponder, std::__ndk1::default_delete<media::mojom::ImageCapture_GetCapabilities_ProxyToResponder> > > >::Destroy(base::internal::BindStateBase const*)+8  /b/build/slave/asan-clang-phone/build/src/base/bind_internal.h:485
  v------>  Release                                                                                                                                                                                                                                                                                                                                                                                                                /b/build/slave/asan-clang-phone/build/src/base/memory/ref_counted.h:572
  000efded  scoped_refptr<base::internal::BindStateBase>::~scoped_refptr()+20                                                                                                                                                                                                                                                                                                                                                      /b/build/slave/asan-clang-phone/build/src/base/memory/ref_counted.h:473
  01a605a3  base::internal::BindState<base::Callback<void (mojo::StructPtr<media::mojom::PhotoCapabilities>), (base::internal::CopyMode)1, (base::internal::RepeatMode)1>, base::internal::PassedWrapper<mojo::StructPtr<media::mojom::PhotoCapabilities> > >::~BindState()+20                                                                                                                                                     /b/build/slave/asan-clang-phone/build/src/base/bind_internal.h:482
  01a6057d  base::internal::BindState<base::Callback<void (mojo::StructPtr<media::mojom::PhotoCapabilities>), (base::internal::CopyMode)1, (base::internal::RepeatMode)1>, base::internal::PassedWrapper<mojo::StructPtr<media::mojom::PhotoCapabilities> > >::Destroy(base::internal::BindStateBase const*)+8                                                                                                                     /b/build/slave/asan-clang-phone/build/src/base/bind_internal.h:485
  v------>  Release                                                                                                                                                                                                                                                                                                                                                                                                                /b/build/slave/asan-clang-phone/build/src/base/memory/ref_counted.h:572
  000efded  scoped_refptr<base::internal::BindStateBase>::~scoped_refptr()+20                                                                                                                                                                                                                                                                                                                                                      /b/build/slave/asan-clang-phone/build/src/base/memory/ref_counted.h:473
  001f4753  base::PendingTask::~PendingTask()+12                                                                                                                                                                                                                                                                                                                                                                                   /b/build/slave/asan-clang-phone/build/src/base/pending_task.cc:42
  001934f1  base::MessageLoop::DeletePendingTasks()+352                                                                                                                                                                                                                                                                                                                                                                            /b/build/slave/asan-clang-phone/build/src/base/message_loop/message_loop.cc:471
  001996f9  base::MessageLoop::~MessageLoop()+596                                                                                                                                                                                                                                                                                                                                                                                  /b/build/slave/asan-clang-phone/build/src/base/message_loop/message_loop.cc:127
  0161b997  base::MessageLoopForUI::~MessageLoopForUI()+6                                                                                                                                                                                                                                                                                                                                                                          /b/build/slave/asan-clang-phone/build/src/base/message_loop/message_loop.h:475
  v------>  operator()                                                                                                                                                                                                                                                                                                                                                                                                             /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/memory:2431
  v------>  reset                                                                                                                                                                                                                                                                                                                                                                                                                  /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/memory:2630
  v------>  ~unique_ptr                                                                                                                                                                                                                                                                                                                                                                                                            /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/memory:2598
  0160c8cd  content::BrowserMainLoop::~BrowserMainLoop()+1976                                                                                                                                                                                                                                                                                                                                                                      /b/build/slave/asan-clang-phone/build/src/content/browser/browser_main_loop.cc:537
  0160cf99  content::BrowserMainLoop::~BrowserMainLoop()+6                                                                                                                                                                                                                                                                                                                                                                         /b/build/slave/asan-clang-phone/build/src/content/browser/browser_main_loop.cc:533
  v------>  operator()                                                                                                                                                                                                                                                                                                                                                                                                             /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/memory:2431
  v------>  reset                                                                                                                                                                                                                                                                                                                                                                                                                  /b/build/slave/asan-clang-phone/build/src/third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include/memory:2630
  0161ed4b  content::BrowserMainRunnerImpl::Shutdown()+1018                                                                                                                                                                                                                                                                                                                                                                        /b/build/slave/asan-clang-phone/build/src/content/browser/browser_main_runner.cc:215
  0160b8b3  content::BrowserMain(content::MainFunctionParams const&)+338                                                                                                                                                                                                                                                                                                                                                           /b/build/slave/asan-clang-phone/build/src/content/browser/browser_main.cc:48
  014d723d  content::BrowserTestBase::SetUp()+1628                                                                                                                                                                                                                                                                                                                                                                                 /b/build/slave/asan-clang-phone/build/src/content/public/test/browser_test_base.cc:260
  01431071  content::ContentBrowserTest::SetUp()+548                                                                                                                                                                                                                                                                                                                                                                               /b/build/slave/asan-clang-phone/build/src/content/public/test/content_browser_test.cc:89
  00eef641  testing::Test::Run()+72                                                                                                                                                                                                                                                                                                                                                                                                /b/build/slave/asan-clang-phone/build/src/third_party/googletest/src/googletest/src/gtest.cc:2467
  00ef04b5  testing::TestInfo::Run()+232                                                                                                                                                                                                                                                                                                                                                                                           /b/build/slave/asan-clang-phone/build/src/third_party/googletest/src/googletest/src/gtest.cc:2653
  00ef100b  testing::TestCase::Run()+258                                                                                                                                                                                                                                                                                                                                                                                           /b/build/slave/asan-clang-phone/build/src/third_party/googletest/src/googletest/src/gtest.cc:2771
  00efaf61  testing::internal::UnitTestImpl::RunAllTests()+956                                                                                                                                                                                                                                                                                                                                                                     /b/build/slave/asan-clang-phone/build/src/third_party/googletest/src/googletest/src/gtest.cc:4648
  00efa9b7  testing::UnitTest::Run()+302                                                                                                                                                                                                                                                                                                                                                                                           /b/build/slave/asan-clang-phone/build/src/third_party/googletest/src/googletest/src/gtest.cc:4256
  016330df  base::TestSuite::Run()+302                                                                                                                                                                                                                                                                                                                                                                                             /b/build/slave/asan-clang-phone/build/src/base/test/test_suite.cc:271
  0143d345  content::ContentTestLauncherDelegate::RunTestSuite(int, char**)+156                                                                                                                                                                                                                                                                                                                                                    /b/build/slave/asan-clang-phone/build/src/content/test/content_test_launcher.cc:105
  01580a59  content::LaunchTests(content::TestLauncherDelegate*, int, int, char**)+748                                                                                                                                                                                                                                                                                                                                             /b/build/slave/asan-clang-phone/build/src/content/public/test/test_launcher.cc:520
  0143d215  main+160                                                                                                                                                                                                                                                                                                                                                                                                               /b/build/slave/asan-clang-phone/build/src/content/test/content_test_launcher.cc:131
  v------>  RunTests                                                                                                                                                                                                                                                                                                                                                                                                               /b/build/slave/asan-clang-phone/build/src/testing/android/native_test/native_test_launcher.cc:131
  00d82a2f  Java_org_chromium_native_1test_NativeTest_nativeRunTests+1498                                                                                                                                                                                                                                                                                                                                                          /b/build/slave/asan-clang-phone/build/src/out/Debug/gen/testing/android/native_test/native_test_jni_headers/testing/jni/NativeTest_jni.h:49
  0001dbcf  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  0004e125  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  00026fe3  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  0002dfa3  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  0002b63b  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  00060863  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  000687c5  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  00026fe3  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  0002dfa3  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  0002b63b  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  0006057f  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  00049d0d  <unknown>                                                                                                                                                                                                                                                                                                                                                                                                              /system/lib/libdvm.so
  0004d2b3  <unknown>  

cc'ing some people who worked on those tests recently.

Comment 2 by awdf@chromium.org, Jun 6 2017

..sorry - intended to link to the build it failed at https://uberchromegw.corp.google.com/i/internal.client.clank/builders/asan-clang-phone/builds/2875 
Cc: mcasas@chromium.org
+ mcasas@

It seems this happens during shutdown. I wonder if this could be related to the Browser main loop shutdown sequence not being aware of and not waiting for video capture internal threads still doing work, see  issue 725271 .
Cc: guidou@chromium.org aleksandar.stojiljkovic@chromium.org
 Issue 734558  has been merged into this issue.
Owner: aleksand...@intel.com
Status: Started (was: Untriaged)
Test is there for since 02.2017.
I understand that it got flaky recently.
johnme@ submitted a skip-workaround https://crrev.com/2944003002 for Android.
I'm not able to access the https://uberchromegw.corp.google.com/* links, and don't see any recent change in master that could trigger this.
Will try to reproduce it locally.

Comment 6 by joh...@chromium.org, Jun 19 2017

Cc: -aleksand...@intel.com
Components: Blink>ImageCapture Blink>MediaStream Tests>Disabled
Labels: -Pri-3 OS-Android Pri-1
> I'm not able to access the https://uberchromegw.corp.google.com/* links

Yeah, unfortunately this is a downstream bot, so the usual trick of replacing https://uberchromegw.corp.google.com/i/ with https://build.chromium.org/p/ doesn't work here :-|

The log output and stacktrace got posted above though, so you're not missing much.

There's another flaky crash in https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.fyi%2FChromium_Mac_10.11%2F9232%2F%2B%2Frecipes%2Fsteps%2Fcontent_browsertests%2F0%2Fstdout which is publicly accessible, though not sure if that one's related to this issue:

[ RUN      ] WebRtcTwoDeviceDepthCaptureBrowserTest.GetBothStreamsAndCheckForFeaturesPresence
[18242:775:0616/134722.632880:12410115371076:INFO:CONSOLE(57)] "Calling getBothStreamsAndCheckForFeaturesPresence", source: http://127.0.0.1:63060/media/getusermedia-depth-capture.html (57)
[18242:775:0616/134722.652747:12410135218941:INFO:CONSOLE(29)] "Test Success", source: http://127.0.0.1:63060/media/webrtc_test_utilities.js (29)
[18242:775:0616/134722.659268:12410141738350:WARNING:ipc_message_attachment_set.cc(49)] MessageAttachmentSet destroyed with unconsumed attachments: 0/1
[18242:775:0616/134722.678236:12410160708465:ERROR:display_link_mac.cc(184)] CVDisplayLinkStop failed: -6672
BrowserTestBase received signal: Segmentation fault: 11. Backtrace:
0   content_browsertests                0x00000001066e210c base::debug::StackTrace::StackTrace(unsigned long) + 28
1   content_browsertests                0x00000001063936c8 content::(anonymous namespace)::DumpStackTraceSignalHandler(int) + 200
2   libsystem_platform.dylib            0x00007fffad208bba _sigtramp + 26
3   ???                                 0x0000000000000016 0x0 + 22
4   content_browsertests                0x000000010731e02e ui::DisplayLinkMac::~DisplayLinkMac() + 14
5   content_browsertests                0x000000010581cbff content::RenderWidgetHostViewMac::~RenderWidgetHostViewMac() + 351
6   content_browsertests                0x000000010581cdfe content::RenderWidgetHostViewMac::~RenderWidgetHostViewMac() + 14
7   libobjc.A.dylib                     0x00007fffac713686 object_cxxDestructFromClass(objc_object*, objc_class*) + 127
8   libobjc.A.dylib                     0x00007fffac70c0c6 objc_destructInstance + 92
9   libobjc.A.dylib                     0x00007fffac70c059 object_dispose + 22
10  AppKit                              0x00007fff955c86e9 -[NSResponder dealloc] + 139
11  AppKit                              0x00007fff955c7102 -[NSView dealloc] + 381
12  content_browsertests                0x0000000107013158 -[BaseView dealloc] + 56
13  content_browsertests                0x0000000105820f48 -[RenderWidgetHostViewCocoa dealloc] + 168
14  libobjc.A.dylib                     0x00007fffac711db7 objc_object::sidetable_release(bool) + 285
15  libobjc.A.dylib                     0x00007fffac70ce8d (anonymous namespace)::AutoreleasePoolPage::pop(void*) + 781
16  CoreFoundation                      0x00007fff97a61856 _CFAutoreleasePoolPop + 22
17  Foundation                          0x00007fff994abf13 -[NSAutoreleasePool drain] + 146
18  content_browsertests                0x00000001077c8aa6 service_manager::Main(service_manager::MainParams const&) + 3142
19  content_browsertests                0x0000000105467a94 content::ContentMain(content::ContentMainParams const&) + 68
20  content_browsertests                0x0000000106393108 content::BrowserTestBase::SetUp() + 1768
21  content_browsertests                0x0000000106383876 content::ContentBrowserTest::SetUp() + 486
22  content_browsertests                0x00000001062f7e11 testing::Test::Run() + 97
23  content_browsertests                0x00000001062f8880 testing::TestInfo::Run() + 288
24  content_browsertests                0x00000001062f8de7 testing::TestCase::Run() + 263
25  content_browsertests                0x00000001062feee7 testing::internal::UnitTestImpl::RunAllTests() + 871
26  content_browsertests                0x00000001062feb53 testing::UnitTest::Run() + 163
27  content_browsertests                0x00000001063bcc43 base::TestSuite::Run() + 163
28  content_browsertests                0x000000010638711a content::ContentTestLauncherDelegate::RunTestSuite(int, char**) + 42
29  content_browsertests                0x00000001063aaa19 content::LaunchTests(content::TestLauncherDelegate*, int, int, char**) + 313
30  content_browsertests                0x00000001063870ba main + 74
31  libdyld.dylib                       0x00007fffacffb255 start + 1

Comment 7 by mcasas@chromium.org, Jun 21 2017

Components: -Blink>ImageCapture
Labels: -Pri-1 Pri-2
I'm not able to reproduce this locally.
Don't see it as a "Priority 1" since there is no depth capture enabled on Android yet.
Reducing to Pri 2 as the test is disabled.
Continuing with attempt to make it reproducible.

While working on Tango support ( Issue 674440 ) I have noticed the crash in compositor, that is not related to this, but plan to add more tests covering it and the case here.

> Labels: -Pri-1 Pri-2
Please, feel free to raise it back to Pri-1 if I misevaluated in #8. 
Thanks Aleksandar, sounds good. [I file P1 bugs by default when disabling tests as sheriff to make sure the bugs get noticed, since it's hard to know how critical the test I'm disabling is, but once feature owners like yourself have looked at the bug I defer to their judgement on prioritisation :-)]
Components: Tests>Disabled
Labels: Test-Disabled

Sign in to add a comment