New issue
Advanced search Search tips

Issue 706163 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

TouchAccessibilityBrowserTest.TouchExplorationInCrossSiteIframe flaky on Linux Tests

Project Member Reported by imch...@chromium.org, Mar 28 2017

Issue description

Logs: https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.linux%2FLinux_Tests%2F53910%2F%2B%2Frecipes%2Fsteps%2Fsite_per_process_content_browsertests%2F0%2Flogs%2FTouchAccessibilityBrowserTest.TouchExplorationInCrossSiteIframe%2F0


TouchAccessibilityBrowserTest.TouchExplorationInCrossSiteIframe (run #1):
[ RUN      ] TouchAccessibilityBrowserTest.TouchExplorationInCrossSiteIframe
[15002:15002:0328/135742.262806:11931378001:WARNING:audio_manager.cc(295)] Multiple instances of AudioManager detected
[15002:15002:0328/135742.262877:11931378052:WARNING:audio_manager.cc(254)] Multiple instances of AudioManager detected
Xlib:  extension "RANDR" missing on display ":99".
[15002:15002:0328/135742.520234:11931635414:WARNING:render_frame_host_impl.cc(2325)] OnDidStopLoading was called twice.
[15002:15002:0328/135742.521679:11931636856:WARNING:render_frame_host_impl.cc(2325)] OnDidStopLoading was called twice.
BrowserTestBase received signal: Terminated. Backtrace:
#0 0x000001e54de7 base::debug::StackTrace::StackTrace()
#1 0x000001b77f86 content::(anonymous namespace)::DumpStackTraceSignalHandler()
#2 0x7fea861dfcb0 <unknown>
#3 0x7fea86295fdd __poll
#4 0x7fea8a599fe4 <unknown>
#5 0x7fea8a59a0ec g_main_context_iteration
#6 0x000001e6d5f6 base::MessagePumpGlib::Run()
#7 0x000001e6b7c7 base::MessageLoop::RunHandler()
#8 0x000001e8540e base::RunLoop::Run()
#9 0x000001b8c2e1 content::MessageLoopRunner::Run()
#10 0x000001b8d451 content::AccessibilityNotificationWaiter::WaitForNotification()
#11 0x0000004f36ca content::TouchAccessibilityBrowserTest_TouchExplorationInCrossSiteIframe_Test::RunTestOnMainThread()
#12 0x000001b6f4f6 content::ContentBrowserTest::RunTestOnMainThreadLoop()
#13 0x000001b77ce8 content::BrowserTestBase::ProxyRunTestOnMainThreadLoop()
#14 0x000001bd06b4 content::ShellBrowserMainParts::PreMainMessageLoopRun()
#15 0x000001720755 content::BrowserMainLoop::PreMainMessageLoopRun()
#16 0x0000019f9046 content::StartupTaskRunner::RunAllTasksNow()
#17 0x00000171e93b content::BrowserMainLoop::CreateStartupTasks()
#18 0x000001723a8c content::BrowserMainRunnerImpl::Initialize()
#19 0x000001bb9795 ShellBrowserMain()
#20 0x000001bb1802 content::ShellMainDelegate::RunProcess()
#21 0x000001670fdc content::RunNamedProcessTypeMain()
#22 0x00000167179c content::ContentMainRunnerImpl::Run()
#23 0x000002936bb5 service_manager::Main()
#24 0x000000d4f0c2 content::ContentMain()
#25 0x000001b77629 content::BrowserTestBase::SetUp()
#26 0x000001b6f3e3 content::ContentBrowserTest::SetUp()
#27 0x000001ce501e testing::Test::Run()
#28 0x000001ce5be0 testing::TestInfo::Run()
#29 0x000001ce5f57 testing::TestCase::Run()
#30 0x000001ced177 testing::internal::UnitTestImpl::RunAllTests()
#31 0x000001cecdf7 testing::UnitTest::Run()
#32 0x000001b95e41 base::TestSuite::Run()
#33 0x000001b72b96 content::ContentTestLauncherDelegate::RunTestSuite()
#34 0x000001b88bea content::LaunchTests()
#35 0x000001b72b43 main
#36 0x7fea861caf45 __libc_start_main
#37 0x0000004c9f2d <unknown>

Suspecting https://chromium.googlesource.com/chromium/src/+/e2da63fad40fc329932c04030bca51f366ed721c

Assigning to kenrb@ for a fix.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 29 2017

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

commit 37401561f9eaf8e8921e3ac4680067dc19ad5d54
Author: kenrb <kenrb@chromium.org>
Date: Wed Mar 29 22:18:47 2017

Fix TouchExplorationInCrossSiteIframe flakiness with OOPIFs enabled

TouchAccessibilityBrowserTest.TouchExplorationInCrossSiteIframe sends
a touch event to a cross-origin iframe. When OOPIFs are enabled, this
relies on the browser hit testing path for the event to reach the
correct RenderWidgetHostView. There is currently flakiness because
sometimes the browser process has not received enough compositor frames
for that hit testing to work.

This patch uses SurfaceHitTestReadyNotifier to delay the event until
hit testing will succeed.

BUG= 706163 

Review-Url: https://codereview.chromium.org/2785793002
Cr-Commit-Position: refs/heads/master@{#460551}

[modify] https://crrev.com/37401561f9eaf8e8921e3ac4680067dc19ad5d54/content/browser/accessibility/touch_accessibility_aura_browsertest.cc

Comment 2 by kenrb@chromium.org, Mar 29 2017

Status: Fixed (was: Untriaged)

Sign in to add a comment