New issue
Advanced search Search tips

Issue 728486 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap flaky on Linux TSan bot

Project Member Reported by pkasting@chromium.org, Jun 1 2017

Issue description

MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap is flakily failing on https://luci-milo.appspot.com/buildbot/chromium.memory/Linux%20TSan%20Tests .

First failure I found was from 5/30: https://luci-milo.appspot.com/buildbot/chromium.memory/Linux%20TSan%20Tests/6710

A more recent failure: https://luci-milo.appspot.com/buildbot/chromium.memory/Linux%20TSan%20Tests/6775

Didn't see any immediate cause of this and the test is fairly new; assigning to the test author for further triage.
 
Hmmm, we're seeing MouseDown, MouseUp, MouseUp, which is pretty strange...

I agree that this problem has probably been there since the test was written, I'll take a look.
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 5 2017

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

commit ecfb2ded05e975b139ea9929958b3b366c65f93e
Author: tdresser <tdresser@chromium.org>
Date: Mon Jun 05 18:36:25 2017

Fix Flaky Mouse latency test.

There are some timing issues that are tricky to get right for cases
where rendering is scheduled due to mouse input. Fix the flake by
preventing rendering from being scheduled.

Mouse down was triggering text selection logic, which scheduled
rendering. To prevent this, use user-select:none.

BUG= 728486 

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

[modify] https://crrev.com/ecfb2ded05e975b139ea9929958b3b366c65f93e/content/browser/renderer_host/input/mouse_latency_browsertest.cc

Status: Fixed (was: Started)
This appears to be fixed.

Sign in to add a comment