New issue
Advanced search Search tips

Issue 799229 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner:
Closed: Jul 10
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

NACL crash on debug builds when toggling off ChromeVox

Project Member Reported by glevin@chromium.org, Jan 4 2018

Issue description

Chrome 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
 
 Issue 799232  has been merged into this issue.
Summary: NACL crash on debug builds when toggling off ChromeVox (was: New SpokenFeedbackTest.ChromeVoxApi test crashing on linux-chromeos-dbg builder)
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.
Project Member

Comment 3 by bugdroid1@chromium.org, 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

 Issue 799322  has been merged into this issue.
Project Member

Comment 6 by bugdroid1@chromium.org, 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

Project Member

Comment 7 by bugdroid1@chromium.org, 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

Project Member

Comment 8 by chromium...@appspot.gserviceaccount.com, Jan 6 2018

Labels: Sheriff-Chromium
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).
Project Member

Comment 9 by bugdroid1@chromium.org, 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

Labels: -Sheriff-Chromium
This has been reverted; removing from sheriff queue.
Cc: hirosh...@chromium.org
 Issue 799520  has been merged into this issue.
Status: wontfix (was: Assigned)

Sign in to add a comment