Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(58)

Issue 2872463002: LayoutTests: Disable flaky tests on Windows. (Closed)

Created:
3 years, 7 months ago by mithro
Modified:
3 years, 7 months ago
CC:
blink-reviews, chromium-reviews
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

LayoutTests: Disable flaky tests on Windows. These tests are failing a small amount of the time on the Windows Random Order bots now they are running sharded on swarming. The tests affected are; * fast/css-grid-layout/grid-self-baseline-two-dimensional.html * fast/css-grid-layout/grid-align-justify-stretch-with-orthogonal-flows.html * fast/dnd/dropEffect-for-effectAllowed.html * external/wpt/css/css-ui-3/outline-004.html Bugs have been filed for each of the failing tests. The most likely cause is that they depend on another test being run and the sharding makes this no longer happening. BUG=717347, 718717, 719298, 719299 NOTRY=true Review-Url: https://codereview.chromium.org/2872463002 Cr-Commit-Position: refs/heads/master@{#469907} Committed: https://chromium.googlesource.com/chromium/src/+/7f6264df1bd3a73c0f8af49adfc345126eebb1cd

Patch Set 1 #

Total comments: 2

Patch Set 2 : LayoutTests: Disable flaky tests on Windows. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
M third_party/WebKit/LayoutTests/TestExpectations View 1 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 17 (11 generated)
mithro
Hi! These tests are failing on the Windows Random Order bots. Tim 'mithro' Ansell
3 years, 7 months ago (2017-05-08 04:22:53 UTC) #6
alancutter (OOO until 2018)
rs lgtm.
3 years, 7 months ago (2017-05-08 04:28:29 UTC) #7
mcgreevy_g
lgtm https://codereview.chromium.org/2872463002/diff/1/third_party/WebKit/LayoutTests/TestExpectations File third_party/WebKit/LayoutTests/TestExpectations (right): https://codereview.chromium.org/2872463002/diff/1/third_party/WebKit/LayoutTests/TestExpectations#newcode3360 third_party/WebKit/LayoutTests/TestExpectations:3360: # Layouts on Swarming (Windows) - https://crbug.com/717347 s/Layouts/LayoutTests/ ...
3 years, 7 months ago (2017-05-08 04:40:59 UTC) #9
mithro
https://codereview.chromium.org/2872463002/diff/1/third_party/WebKit/LayoutTests/TestExpectations File third_party/WebKit/LayoutTests/TestExpectations (right): https://codereview.chromium.org/2872463002/diff/1/third_party/WebKit/LayoutTests/TestExpectations#newcode3360 third_party/WebKit/LayoutTests/TestExpectations:3360: # Layouts on Swarming (Windows) - https://crbug.com/717347 On 2017/05/08 ...
3 years, 7 months ago (2017-05-08 04:48:11 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2872463002/20001
3 years, 7 months ago (2017-05-08 04:48:37 UTC) #14
commit-bot: I haz the power
3 years, 7 months ago (2017-05-08 04:56:28 UTC) #17
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/7f6264df1bd3a73c0f8af49adfc3...

Powered by Google App Engine
This is Rietveld 408576698