New issue
Advanced search Search tips

Issue 892097 link

Starred by 1 user

Issue metadata

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

Blocking:
issue 874089



Sign in to add a comment

TouchActionBrowserTest.PanXYAtXAreaMainThreadJanky is flaky

Project Member Reported by Findit, Oct 4

Issue description


Flaky test: TouchActionBrowserTest.PanXYAtXAreaMainThreadJanky
Sample failed build due to flakiness: https://chromium-swarm.appspot.com/task?id=405317dc7c793710
Test output log: https://ci.chromium.org/buildbot/chromium.chromiumos/linux-chromeos-rel/14176
Culprit (90.1% confidence): https://chromium-review.googlesource.com/q/I1bbf82170adf0e254eda8ffa7f1804d56495cb2a
Analysis: https://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVy0QELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCKaAWNocm9taXVtLmNocm9taXVtb3MvbGludXgtY2hyb21lb3MtcmVsLzE0MTc2L3NpbmdsZV9wcm9jZXNzX21hc2hfY29udGVudF9icm93c2VydGVzdHMvVkc5MVkyaEJZM1JwYjI1Q2NtOTNjMlZ5VkdWemRDNVFZVzVZV1VGMFdFRnlaV0ZOWVdsdVZHaHlaV0ZrU21GdWEzaz0MCxITTWFzdGVyRmxha2VBbmFseXNpcxgBDA

Please revert the culprit, or disable the test and find the appropriate owner.

If the culprit above is wrong, please file a bug using this link:
https://bugs.chromium.org/p/chromium/issues/entry?status=Unconfirmed&labels=Pri-1,Test-Findit-Wrong&components=Tools%3ETest%3EFindit%3EFlakiness&summary=%5BFindit%5D%20Flake%20Analyzer%20-%20Wrong%20result%20for%20TouchActionBrowserTest.PanXYAtXAreaMainThreadJanky&comment=Link%20to%20Analysis%3A%20https://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVy0QELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCKaAWNocm9taXVtLmNocm9taXVtb3MvbGludXgtY2hyb21lb3MtcmVsLzE0MTc2L3NpbmdsZV9wcm9jZXNzX21hc2hfY29udGVudF9icm93c2VydGVzdHMvVkc5MVkyaEJZM1JwYjI1Q2NtOTNjMlZ5VkdWemRDNVFZVzVZV1VGMFdFRnlaV0ZOWVdsdVZHaHlaV0ZrU21GdWEzaz0MCxITTWFzdGVyRmxha2VBbmFseXNpcxgBDA
 
Components: Blink>Input
Blocking: 874089
Labels: Proj-Mash-SingleProcess
Owner: xiy...@chromium.org
Status: Assigned (was: Untriaged)
[ RUN      ] TouchActionBrowserTest.PanXYAtXAreaMainThreadJanky
[8782:8789:1003/190238.256768:ERROR:network_service.cc(88)] Not implemented reached in std::unique_ptr<net::NetworkChangeNotifier> network::(anonymous namespace)::CreateNetworkChangeNotifierIfNeeded()
[8782:8782:1003/190238.281996: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:42797/devtools/browser/d5ebc185-cd88-4e30-b47f-c43c07ab799f
[8782:8782:1003/190238.282737: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.
[8782:8782:1003/190238.287700:ERROR:render_widget_host_view_aura.cc(1278)] Not implemented reached in virtual base::i18n::TextDirection content::RenderWidgetHostViewAura::GetTextDirection() const
[8782:8782:1003/190238.320333:ERROR:layer_tree_host_impl.cc(3098)] Forcing zero-copy tile initialization as worker context is missing
BrowserTestBase received signal: Segmentation fault. Backtrace:
#0 0x0000029157fc base::debug::StackTrace::StackTrace()
#1 0x00000255ff85 content::(anonymous namespace)::DumpStackTraceSignalHandler()
#2 0x7f518f865cb0 <unknown>
#3 0x000000c2bc89 device::mojom::PowerMonitorClientProxy::PowerStateChange()
#4 0x000000939079 content::TouchActionBrowserTest::LoadURL()
#5 0x00000093a4d8 content::TouchActionBrowserTest_PanXYAtXAreaMainThreadJanky_Test::RunTestOnMainThread()
#6 0x00000255f44b content::BrowserTestBase::ProxyRunTestOnMainThreadLoop()
#7 0x0000025ca838 content::ShellBrowserMainParts::PreMainMessageLoopRun()
#8 0x000001fa4ba1 content::BrowserMainLoop::PreMainMessageLoopRun()
#9 0x00000231d237 content::StartupTaskRunner::RunAllTasksNow()
#10 0x000001fa351d content::BrowserMainLoop::CreateStartupTasks()
#11 0x000001fa7043 content::BrowserMainRunnerImpl::Initialize()
#12 0x0000025b20c9 ShellBrowserMain()
#13 0x0000025a8782 content::ShellMainDelegate::RunProcess()
#14 0x000001ed5f7d content::ContentMainRunnerImpl::Run()
#15 0x000003bf5790 service_manager::Main()
#16 0x000001636f64 content::ContentMain()
#17 0x00000255f081 content::BrowserTestBase::SetUp()
#18 0x0000013f233d testing::Test::Run()
#19 0x0000013f2f60 testing::TestInfo::Run()
#20 0x0000013f3477 testing::TestCase::Run()
#21 0x0000013ff1e7 testing::internal::UnitTestImpl::RunAllTests()
#22 0x0000013fed5d testing::UnitTest::Run()
#23 0x00000258de41 base::TestSuite::Run()
#24 0x000002558cca content::ContentTestLauncherDelegate::RunTestSuite()
#25 0x0000025775fe content::LaunchTests()
#26 0x000002558c90 main
#27 0x7f518f850f45 __libc_start_main
#28 0x0000005964ca _start
[1/1] TouchActionBrowserTest.PanXYAtXAreaMainThreadJanky (CRASHED)
Project Member

Comment 4 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.
Turns out the crash is because RenderFrameMetadataProviderImpl::ReportAllFrameSubmissionsForTesting is called before Bind.

Linker has merged ReportAllFrameSubmissionsForTesting proxy impl with PowerStateChange.
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