New issue
Advanced search Search tips

Issue 892098 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 9
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug
Flaky-Test: TouchActionBrowserTest.TouchActionNone

Blocking:
issue 874089



Sign in to add a comment

TouchActionBrowserTest.TouchActionNone is flaky

Project Member Reported by Findit, Oct 4

Issue description

Components: Blink>Input
This test had already been disabled in many cases:

#if !defined(NDEBUG) || defined(ADDRESS_SANITIZER) ||       \
    defined(MEMORY_SANITIZER) || defined(LEAK_SANITIZER) || \
    defined(THREAD_SANITIZER)

I guess we should just disable it completely...
Blocking: 874089
Labels: Proj-Mash-SingleProcess
Owner: xiy...@chromium.org
Status: Assigned (was: Untriaged)
This is failure in single process mash. I will disable it first and check it out later on.
Crash stack of failed run:

[ RUN      ] TouchActionBrowserTest.TouchActionNone
[18030:18037:1004/041951.335090:ERROR:network_service.cc(88)] Not implemented reached in std::unique_ptr<net::NetworkChangeNotifier> network::(anonymous namespace)::CreateNetworkChangeNotifierIfNeeded()
[18030:18030:1004/041951.355813:ERROR:gpu_interface_provider.cc(87)] Not implemented reached in virtual void content::GpuInterfaceProvider::RegisterOzoneGpuInterfaces(service_manager::BinderRegistry *)

DevTools listening on ws://127.0.0.1:33610/devtools/browser/d25f41b1-1a3e-4060-acf2-31385c586adf
[18030:18030:1004/041951.356872:ERROR:proxy_resolution_service.cc(1555)] ProxyConfigService for ChromeOS should be created in profile_io_data.cc::CreateProxyConfigService and this should be used only for examples.
[18030:18030:1004/041951.362797:ERROR:render_widget_host_view_aura.cc(1278)] Not implemented reached in virtual base::i18n::TextDirection content::RenderWidgetHostViewAura::GetTextDirection() const
[18030:18030:1004/041951.402817:ERROR:layer_tree_host_impl.cc(3098)] Forcing zero-copy tile initialization as worker context is missing
BrowserTestBase received signal: Segmentation fault. Backtrace:
#0 0x000002916a1c base::debug::StackTrace::StackTrace()
#1 0x000002561195 content::(anonymous namespace)::DumpStackTraceSignalHandler()
#2 0x7fef489fecb0 <unknown>
#3 0x000000c2bbc9 device::mojom::PowerMonitorClientProxy::PowerStateChange()
#4 0x000000938fb9 content::TouchActionBrowserTest::LoadURL()
#5 0x00000093984a content::TouchActionBrowserTest_TouchActionNone_Test::RunTestOnMainThread()
#6 0x00000256065b content::BrowserTestBase::ProxyRunTestOnMainThreadLoop()
#7 0x0000025cba48 content::ShellBrowserMainParts::PreMainMessageLoopRun()
#8 0x000001fa5dc1 content::BrowserMainLoop::PreMainMessageLoopRun()
#9 0x00000231e447 content::StartupTaskRunner::RunAllTasksNow()
#10 0x000001fa473d content::BrowserMainLoop::CreateStartupTasks()
#11 0x000001fa8263 content::BrowserMainRunnerImpl::Initialize()
#12 0x0000025b32d9 ShellBrowserMain()
#13 0x0000025a9992 content::ShellMainDelegate::RunProcess()
#14 0x000001ed719d content::ContentMainRunnerImpl::Run()
#15 0x000003bf64e0 service_manager::Main()
#16 0x000001636dc4 content::ContentMain()
#17 0x000002560291 content::BrowserTestBase::SetUp()
#18 0x0000013f215d testing::Test::Run()
#19 0x0000013f2d80 testing::TestInfo::Run()
#20 0x0000013f3297 testing::TestCase::Run()
#21 0x0000013ff007 testing::internal::UnitTestImpl::RunAllTests()
#22 0x0000013feb7d testing::UnitTest::Run()
#23 0x00000258f051 base::TestSuite::Run()
#24 0x000002559eda content::ContentTestLauncherDelegate::RunTestSuite()
#25 0x00000257880e content::LaunchTests()
#26 0x000002559ea0 main
#27 0x7fef489e9f45 __libc_start_main
#28 0x00000059640a _start
[1/1] TouchActionBrowserTest.TouchActionNone (CRASHED)
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6a703e783ff3c2912fb486056d466757ebae7d6a

commit 6a703e783ff3c2912fb486056d466757ebae7d6a
Author: Xiyuan Xia <xiyuan@chromium.org>
Date: Thu Oct 04 15:25:44 2018

Disable two tests in single process mash content_browsertests

Occasionally crash:
  TouchActionBrowserTest.PanXYAtXAreaMainThreadJanky
  TouchActionBrowserTest.TouchActionNone

TBR=jamescook@chromium.org

Bug:  892097 ,  892098 
Change-Id: Icc26e7c5e9dfe632447e185a9567d17f0bd404a9
Reviewed-on: https://chromium-review.googlesource.com/c/1261401
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596679}
[modify] https://crrev.com/6a703e783ff3c2912fb486056d466757ebae7d6a/testing/buildbot/filters/chromeos.single_process_mash.content_browsertests.filter

Labels: -Sheriff-Chromium
Removing Sheriff-Chromium as the test is now disabled.
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 9

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f243fb861e6f4343dcab02db4a6f52c5af38b011

commit f243fb861e6f4343dcab02db4a6f52c5af38b011
Author: Xiyuan Xia <xiyuan@chromium.org>
Date: Tue Oct 09 19:31:49 2018

Handle premature ReportAllFrameSubmissionsForTesting call

RFMPImpl::ReportAllFrameSubmissionsForTesting could be called
before its Bind and causes test to crash. The CL fixes the
issue by deferring the call until Bind.

Bug:  892097 ,  892098 
Test: TouchActionBrowserTest.PanXYAtXAreaMainThreadJanky does not crash.
Change-Id: If7ad5303dfd162b4c1dfb51cb7a77ed19d6ca8cb
Reviewed-on: https://chromium-review.googlesource.com/c/1263043
Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#598038}
[modify] https://crrev.com/f243fb861e6f4343dcab02db4a6f52c5af38b011/content/browser/renderer_host/render_frame_metadata_provider_impl.cc
[modify] https://crrev.com/f243fb861e6f4343dcab02db4a6f52c5af38b011/content/browser/renderer_host/render_frame_metadata_provider_impl.h
[modify] https://crrev.com/f243fb861e6f4343dcab02db4a6f52c5af38b011/testing/buildbot/filters/chromeos.single_process_mash.content_browsertests.filter

Status: Fixed (was: Assigned)

Sign in to add a comment