New issue
Advanced search Search tips

Issue 736836 link

Starred by 1 user
Status: Fixed
Owner:
tdres...@chromium.org
Closed: Oct 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap is flaky on Linux

Project Member Reported by imch...@chromium.org, Jun 26 2017 
(Please move to another component if needed. Thanks!)

Disabling this test on Linux for now.

Flakiness dashboard: https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=browser_side_navigation_content_browsertests&tests=MouseDownAndUpRecordedWithoutSwap


Logs from recent failure:

[ RUN      ] MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap
Xlib:  extension "RANDR" missing on display ":99".
[8511:8574:0626/105340.121724:26870220798:ERROR:devtools_http_handler.cc(804)]
DevTools listening on 127.0.0.1:59929
[8573:8629:0626/105341.140330:26871239245:WARNING:internal_linux.cc(64)] Failed to read /proc/self/stat
[1:3:0626/105341.140251:26871239417:WARNING:internal_linux.cc(64)] Failed to read /proc/self/stat
../../content/browser/renderer_host/input/mouse_latency_browsertest.cc:188: Failure
Value of: trace_event_names
Expected: has 4 elements where
element #0 is equal to "InputLatency::MouseDown",
element #1 is equal to "InputLatency::MouseDown",
element #2 is equal to "InputLatency::MouseUp",
element #3 is equal to "InputLatency::MouseUp"
  Actual: { "InputLatency::MouseDown", "InputLatency::MouseUp", "InputLatency::MouseDown", "InputLatency::MouseUp" }, whose element #1 doesn't match
[  FAILED  ] MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap, where TypeParam =  and GetParam() =  (1766 ms)
[ RUN      ] MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap
Xlib:  extension "RANDR" missing on display ":99".
[23602:23630:0626/110632.495698:27642594648:ERROR:devtools_http_handler.cc(804)]
DevTools listening on 127.0.0.1:46387
[23628:23644:0626/110633.373699:27643472606:WARNING:internal_linux.cc(64)] Failed to read /proc/self/stat
[1:3:0626/110633.373861:27643473038:WARNING:internal_linux.cc(64)] Failed to read /proc/self/stat
../../content/browser/renderer_host/input/mouse_latency_browsertest.cc:188: Failure
Value of: trace_event_names
Expected: has 4 elements where
element #0 is equal to "InputLatency::MouseDown",
element #1 is equal to "InputLatency::MouseDown",
element #2 is equal to "InputLatency::MouseUp",
element #3 is equal to "InputLatency::MouseUp"
  Actual: { "InputLatency::MouseDown", "InputLatency::MouseUp", "InputLatency::MouseDown", "InputLatency::MouseUp" }, whose element #1 doesn't match
[  FAILED  ] MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap, where TypeParam =  and GetParam() =  (1529 ms)
[ RUN      ] MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap
Xlib:  extension "RANDR" missing on display ":99".
[23901:23929:0626/110645.410418:27655509355:ERROR:devtools_http_handler.cc(804)]
DevTools listening on 127.0.0.1:35136
[23927:23943:0626/110646.314861:27656413785:WARNING:internal_linux.cc(64)] Failed to read /proc/self/stat
[1:3:0626/110646.314959:27656414140:WARNING:internal_linux.cc(64)] Failed to read /proc/self/stat
../../content/browser/renderer_host/input/mouse_latency_browsertest.cc:188: Failure
Value of: trace_event_names
Expected: has 4 elements where
element #0 is equal to "InputLatency::MouseDown",
element #1 is equal to "InputLatency::MouseDown",
element #2 is equal to "InputLatency::MouseUp",
element #3 is equal to "InputLatency::MouseUp"
  Actual: { "InputLatency::MouseDown", "InputLatency::MouseUp", "InputLatency::MouseDown", "InputLatency::MouseUp" }, whose element #1 doesn't match
[  FAILED  ] MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap, where TypeParam =  and GetParam() =  (1591 ms)
[ RUN      ] MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap
Xlib:  extension "RANDR" missing on display ":99".
[23947:23975:0626/110647.847956:27657946896:ERROR:devtools_http_handler.cc(804)]
DevTools listening on 127.0.0.1:52286
[23973:23989:0626/110648.743699:27658842618:WARNING:internal_linux.cc(64)] Failed to read /proc/self/stat
[1:3:0626/110648.743909:27658843081:WARNING:internal_linux.cc(64)] Failed to read /proc/self/stat
../../content/browser/renderer_host/input/mouse_latency_browsertest.cc:188: Failure
Value of: trace_event_names
Expected: has 4 elements where
element #0 is equal to "InputLatency::MouseDown",
element #1 is equal to "InputLatency::MouseDown",
element #2 is equal to "InputLatency::MouseUp",
element #3 is equal to "InputLatency::MouseUp"
  Actual: { "InputLatency::MouseDown", "InputLatency::MouseUp", "InputLatency::MouseDown", "InputLatency::MouseUp" }, whose element #1 doesn't match
[  FAILED  ] MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap, where TypeParam =  and GetParam() =  (1595 ms)
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 26 2017 

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

commit 06c25166087d1935075d2e95e2435f6bc7b7ee51
Author: imcheng <imcheng@chromium.org>
Date: Mon Jun 26 18:43:36 2017

Disable flaky test MouseLatencyBrowserTest.MouseDownAndUpRecordedWithoutSwap on Linux.

BUG= 736836 
NOTRY=true
TBR=xidachen@chromium.org

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

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

Comment 2 by tdres...@chromium.org, Oct 4 2017

Components: -UI>Input Speed>Metrics
Labels: Hotlist-Input-Dev
Owner: tdres...@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 4 2017 

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

commit 673559d721ae8f3524e41d7ec94f5448ff50e5ca
Author: Tim Dresser <tdresser@chromium.org>
Date: Wed Oct 04 18:21:35 2017

Deflake and enable MouseLatencyBrowserTest on Linux.

There was previously a rare race condition in which trace events would
arrive in a different order, breaking this test.

We don't actually care about the ordering of these events, so we no
longer verify their order.

Bug:  736836 
Change-Id: Iabb65f603c7fbb985faa97f43a65a5313711c53f
Reviewed-on: https://chromium-review.googlesource.com/700714
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Commit-Queue: Timothy Dresser <tdresser@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506449}
[modify] https://crrev.com/673559d721ae8f3524e41d7ec94f5448ff50e5ca/content/browser/renderer_host/input/mouse_latency_browsertest.cc

Comment 4 by tdres...@chromium.org, Oct 4 2017

Status: Fixed (was: Started)
►

Sign in to add a comment