New issue
Advanced search Search tips

Issue 707491 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

"MainThreadEventQueueBrowserTest.MouseMove" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Apr 1 2017

Issue description

"MainThreadEventQueueBrowserTest.MouseMove" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyNAsSBUZsYWtlIilNYWluVGhyZWFkRXZlbnRRdWV1ZUJyb3dzZXJUZXN0Lk1vdXNlTW92ZQw.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 
Components: Blink>Input
Labels: Hotlist-Input-Dev
Status: Started (was: Assigned)
Project Member

Comment 3 by chromium...@appspot.gserviceaccount.com, Apr 3 2017

Detected 3 new flakes for test/step "MainThreadEventQueueBrowserTest.MouseMove". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyNAsSBUZsYWtlIilNYWluVGhyZWFkRXZlbnRRdWV1ZUJyb3dzZXJUZXN0Lk1vdXNlTW92ZQw. This message was posted automatically by the chromium-try-flakes app.
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 4 2017

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

commit f5cd7b09e1c1b61047a21055e24537365cb04b87
Author: maxmorin <maxmorin@chromium.org>
Date: Tue Apr 04 10:38:04 2017

Disable flaky MainThreadEventQueueBrowserTest.MouseMove.

TBR=dtapuska@chromium.org
BUG= 707491 

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

[modify] https://crrev.com/f5cd7b09e1c1b61047a21055e24537365cb04b87/content/browser/renderer_host/input/main_thread_event_queue_browsertest.cc

Labels: -Sheriff-Chromium

Comment 6 by st...@chromium.org, Apr 4 2017

Labels: -AnalyzedByFindit Test-Findit-Analyzed
Project Member

Comment 7 by bugdroid1@chromium.org, Apr 4 2017

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

commit b1e592331aa12efbde84280e8b4e9248a7360c9d
Author: dtapuska <dtapuska@chromium.org>
Date: Tue Apr 04 21:21:56 2017

Allow events in queue to coalesce while executing other events.

The code removed all events from the queue at the start which is flawed
because a slow handler could cause another event not to be coalesced.
This was explicitly tested in the browser test that is failing sometimes
now.

BUG= 707491 , 674301 

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

[modify] https://crrev.com/b1e592331aa12efbde84280e8b4e9248a7360c9d/content/browser/renderer_host/input/main_thread_event_queue_browsertest.cc
[modify] https://crrev.com/b1e592331aa12efbde84280e8b4e9248a7360c9d/content/renderer/input/main_thread_event_queue.cc
[modify] https://crrev.com/b1e592331aa12efbde84280e8b4e9248a7360c9d/content/renderer/input/main_thread_event_queue_task_list.h

Status: Fixed (was: Started)

Sign in to add a comment