Issue metadata
Sign in to add a comment
|
NACL crash on debug builds when toggling off ChromeVox |
||||||||||||||||||||||||
Issue descriptionChrome Version: 65.0.3312.0 OS: Chrome The new SpokenFeedbackTest.ChromeVoxApi test added in https://chromium-review.googlesource.com/c/chromium/src/+/847896 is crashing on the linux-chromeos-dbg builder (https://ci.chromium.org/buildbot/chromium.chromiumos/linux-chromeos-dbg/3567). I'm going to revert the CL. Here's the dump from the test logs: [0104/200431.227510:FATAL:ref_counted.h(61)] Check failed: CalledOnValidSequence(). [0104/200431.227580:ERROR:debugger_posix.cc(174)] Not implemented reached in bool base::debug::BeingDebugged() ** abort() called ** Signal 4 from untrusted code: pc=59bd0ffbe7c0 [28042:28069:0104/120431.230859:ERROR:nacl_process_host.cc(256)] NaCl process exited with status 64512 (0xfc00) BrowserTestBase received signal: Terminated. Backtrace: #0 0x7fa04d9d7dad base::debug::StackTrace::StackTrace() #1 0x7fa04d9d632c base::debug::StackTrace::StackTrace() #2 0x0000047b34ee content::(anonymous namespace)::DumpStackTraceSignalHandler() #3 0x7fa031e7fcb0 <unknown> #4 0x7fa031f47693 epoll_wait #5 0x7fa04dd33bc1 epoll_dispatch #6 0x7fa04dd271b5 event_base_loop #7 0x7fa04da87d92 base::MessagePumpLibevent::Run() #8 0x7fa04da80c6c base::MessageLoop::Run() #9 0x7fa04db35ac5 base::RunLoop::Run() #10 0x0000047ed4e6 content::RunThisRunLoop() #11 0x0000047eebb8 content::MessageLoopRunner::Run() #12 0x0000038fc938 chromeos::SpeechMonitor::GetNextUtterance() #13 0x0000009a4ce9 chromeos::SpokenFeedbackTest_ChromeVoxApi_Test::RunTestOnMainThread() #14 0x0000047b3067 content::BrowserTestBase::ProxyRunTestOnMainThreadLoop() #15 0x0000006678bd _ZN4base8internal13FunctorTraitsIMN25WebViewImeInteractiveTest30CompositionRangeUpdateObserverEFvvEvE6InvokeIPS3_JEEEvS5_OT_DpOT0_ #16 0x000000667804 _ZN4base8internal12InvokeHelperILb0EvE8MakeItSoIMN26WebViewInteractiveTestBase20PopupCreatedObserverEFvvEJPS5_EEEvOT_DpOT0_ #17 0x0000047b5585 _ZN4base8internal7InvokerINS0_9BindStateIMN7content15BrowserTestBaseEFvvEJNS0_17UnretainedWrapperIS4_EEEEEFvvEE7RunImplIRKS6_RKNSt3__15tupleIJS8_EEEJLm0EEEEvOT_OT0_NSF_16integer_sequenceImJXspT1_EEEE #18 0x0000047b54cc _ZN4base8internal7InvokerINS0_9BindStateIMN7content15BrowserTestBaseEFvvEJNS0_17UnretainedWrapperIS4_EEEEEFvvEE3RunEPNS0_13BindStateBaseE #19 0x0000006bad6d _ZNKR4base17RepeatingCallbackIFvvEE3RunEv #20 0x000003940de0 ChromeBrowserMainParts::PreMainMessageLoopRunImpl() #21 0x00000393f15c ChromeBrowserMainParts::PreMainMessageLoopRun() #22 0x0000026908ce chromeos::ChromeBrowserMainPartsChromeos::PreMainMessageLoopRun() #23 0x7fa048db3a34 content::BrowserMainLoop::PreMainMessageLoopRun() #24 0x7fa047c27fdd _ZN4base8internal13FunctorTraitsIMN7content14NetworkContextEFvvEvE6InvokeIPS3_JEEEvS5_OT_DpOT0_ #25 0x7fa047c27f24 _ZN4base8internal12InvokeHelperILb0EvE8MakeItSoIMN7content14NetworkContextEFvvEJPS5_EEEvOT_DpOT0_ #26 0x7fa048dba985 _ZN4base8internal7InvokerINS0_9BindStateIMN7content15BrowserMainLoopEFivEJNS0_17UnretainedWrapperIS4_EEEEEFivEE7RunImplIRKS6_RKNSt3__15tupleIJS8_EEEJLm0EEEEiOT_OT0_NSF_16integer_sequenceImJXspT1_EEEE #27 0x7fa048dba8cc _ZN4base8internal7InvokerINS0_9BindStateIMN7content15BrowserMainLoopEFivEJNS0_17UnretainedWrapperIS4_EEEEEFivEE3RunEPNS0_13BindStateBaseE #28 0x7fa047bef43d _ZNKR4base17RepeatingCallbackIFvvEE3RunEv #29 0x7fa049b2a8ed content::StartupTaskRunner::RunAllTasksNow() #30 0x7fa048daef69 content::BrowserMainLoop::CreateStartupTasks() #31 0x7fa048dbc95d content::BrowserMainRunnerImpl::Initialize() #32 0x7fa048da7d94 content::BrowserMain() #33 0x7fa04ac662ed content::RunNamedProcessTypeMain() #34 0x7fa04ac68d5a content::ContentMainRunnerImpl::Run() #35 0x7fa04ac6045d content::ContentServiceManagerMainDelegate::RunEmbedderProcess() #36 0x7fa043f56fdc service_manager::Main() #37 0x7fa04ac6521b content::ContentMain() #38 0x0000047b2cc0 content::BrowserTestBase::SetUp() #39 0x000003902c5e InProcessBrowserTest::SetUp() #40 0x000000a9fcbe testing::internal::HandleSehExceptionsInMethodIfSupported<>() #41 0x000000a92f52 testing::internal::HandleExceptionsInMethodIfSupported<>() #42 0x000000a7e084 testing::Test::Run() #43 0x000000a7ea0d testing::TestInfo::Run() #44 0x000000a7f46c testing::TestCase::Run() #45 0x000000a8afcc testing::internal::UnitTestImpl::RunAllTests() #46 0x000000aa06de testing::internal::HandleSehExceptionsInMethodIfSupported<>() #47 0x000000a945b2 testing::internal::HandleExceptionsInMethodIfSupported<>() #48 0x000000a8abde testing::UnitTest::Run() #49 0x0000049b6671 RUN_ALL_TESTS() #50 0x0000049b3742 base::TestSuite::Run() #51 0x000000855feb InteractiveUITestSuiteRunner::RunTestSuite() #52 0x0000038e31b9 ChromeTestLauncherDelegate::RunTestSuite() #53 0x0000047e3218 content::LaunchTests() #54 0x0000038e3ecf LaunchChromeTests() #55 0x000000855e7d main #56 0x7fa031e6af45 __libc_start_main #57 0x00000062298a _start
,
Jan 4 2018
Not sure if there's anything to do since this was reverted. I'm relanding and disabling on debug builds. The crash existed prior to the change (which contains only the test). I'll repurpose this bug to track.
,
Jan 5 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a971bf671716874136312d1dace4bb91a9169ff4 commit a971bf671716874136312d1dace4bb91a9169ff4 Author: David Tseng <dtseng@chromium.org> Date: Fri Jan 05 00:18:45 2018 Reland: Add test coverage for ChromeVox api Excludes this test on debug builds. This change only contains a test; the NACL crash existed prior to this change. Original change: https://chromium-review.googlesource.com/c/chromium/src/+/847896 Bug: 799229 TBR=dtseng@chromium.org Change-Id: I0060686b6f1e35d756672597c6a95f4f161fa318 Reviewed-on: https://chromium-review.googlesource.com/851262 Reviewed-by: David Tseng <dtseng@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#527161} [modify] https://crrev.com/a971bf671716874136312d1dace4bb91a9169ff4/chrome/browser/chromeos/accessibility/spoken_feedback_browsertest.cc
,
Jan 5 2018
According to the flakiness dashboard https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=interactive_ui_tests&tests=TestAsNormalAndGuestUser%2FSpokenFeedbackTest.ChromeVoxApi the test was also failing on ChromeOS release bots.
,
Jan 5 2018
Issue 799322 has been merged into this issue.
,
Jan 5 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a735c37f75407375eb9ec86027d348e76acba542 commit a735c37f75407375eb9ec86027d348e76acba542 Author: Hiroshige Hayashizaki <hiroshige@chromium.org> Date: Fri Jan 05 04:26:56 2018 Revert "Reland: Add test coverage for ChromeVox api" This reverts commit a971bf671716874136312d1dace4bb91a9169ff4. Reason for revert: The test failing on ChromeOS LSAN/MSAN bots. https://uberchromegw.corp.google.com/i/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/25547 https://uberchromegw.corp.google.com/i/chromium.memory/builders/Linux%20ChromiumOS%20MSan%20Tests/builds/5069 Original change's description: > Reland: Add test coverage for ChromeVox api > > Excludes this test on debug builds. This change only contains a test; the NACL crash existed prior to this change. > > Original change: > https://chromium-review.googlesource.com/c/chromium/src/+/847896 > > Bug: 799229 > TBR=dtseng@chromium.org > > Change-Id: I0060686b6f1e35d756672597c6a95f4f161fa318 > Reviewed-on: https://chromium-review.googlesource.com/851262 > Reviewed-by: David Tseng <dtseng@chromium.org> > Commit-Queue: David Tseng <dtseng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#527161} TBR=dtseng@chromium.org Change-Id: I2acfb8c31d7d18bb1e57e204694cee25a2e8ba8f No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 799229 Reviewed-on: https://chromium-review.googlesource.com/851632 Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org> Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Cr-Commit-Position: refs/heads/master@{#527198} [modify] https://crrev.com/a735c37f75407375eb9ec86027d348e76acba542/chrome/browser/chromeos/accessibility/spoken_feedback_browsertest.cc
,
Jan 5 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f3e04053fdc5e1bba109fc0a0b908d5e73667930 commit f3e04053fdc5e1bba109fc0a0b908d5e73667930 Author: David Tseng <dtseng@chromium.org> Date: Fri Jan 05 23:43:36 2018 Reland2: Add test coverage for ChromeVox api Excludes this test on debug builds. This change only contains a test; the NACL crash existed prior to this change. Original change: https://chromium-review.googlesource.com/c/chromium/src/+/847896 Bug: 799229 TBR=dtseng@chromium.org Change-Id: Ic638b8a7a2b140718402e7618e3909054fd2d3a7 Reviewed-on: https://chromium-review.googlesource.com/852772 Reviewed-by: David Tseng <dtseng@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#527428} [modify] https://crrev.com/f3e04053fdc5e1bba109fc0a0b908d5e73667930/chrome/browser/chromeos/accessibility/spoken_feedback_browsertest.cc
,
Jan 6 2018
Detected 3 new flakes for test/step "TestAsNormalAndGuestUser/SpokenFeedbackTest.ChromeVoxApi/1". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyRQsSBUZsYWtlIjpUZXN0QXNOb3JtYWxBbmRHdWVzdFVzZXIvU3Bva2VuRmVlZGJhY2tUZXN0LkNocm9tZVZveEFwaS8xDA. This message was posted automatically by the chromium-try-flakes app. Since flakiness is ongoing, the issue was moved back into Sheriff Bug Queue (unless already there).
,
Jan 6 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e971a6192a4ae96cd82e7ddedee927a790e36be2 commit e971a6192a4ae96cd82e7ddedee927a790e36be2 Author: David Tseng <dtseng@chromium.org> Date: Sat Jan 06 02:28:10 2018 Revert "Reland2: Add test coverage for ChromeVox api" This reverts commit f3e04053fdc5e1bba109fc0a0b908d5e73667930. Reason for revert: <INSERT REASONING HERE> Original change's description: > Reland2: Add test coverage for ChromeVox api > > Excludes this test on debug builds. This change only contains a test; the NACL crash existed prior to this change. > > Original change: > https://chromium-review.googlesource.com/c/chromium/src/+/847896 > > Bug: 799229 > TBR=dtseng@chromium.org > > Change-Id: Ic638b8a7a2b140718402e7618e3909054fd2d3a7 > Reviewed-on: https://chromium-review.googlesource.com/852772 > Reviewed-by: David Tseng <dtseng@chromium.org> > Commit-Queue: David Tseng <dtseng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#527428} TBR=dtseng@chromium.org Change-Id: I94cd5965ffeb9095815bbf29c55b67b7e368e697 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 799229Looks like possibly flakes in guest mode. Unsure why but best to pull this for now. Reviewed-on: https://chromium-review.googlesource.com/852906 Reviewed-by: David Tseng <dtseng@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#527475} [modify] https://crrev.com/e971a6192a4ae96cd82e7ddedee927a790e36be2/chrome/browser/chromeos/accessibility/spoken_feedback_browsertest.cc
,
Jan 8 2018
This has been reverted; removing from sheriff queue.
,
Jan 9 2018
,
Jul 10
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by hirosh...@chromium.org
, Jan 4 2018