New issue
Advanced search Search tips

Issue 816868 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 817030
Owner: ----
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Lots of flaky external/wpt/css/css-text/ tests

Project Member Reported by hbos@chromium.org, Feb 27 2018

Issue description

Observed flakiness on:

external/wpt/css/css-text/text-align/text-align-end-007.html
external/wpt/css/css-text/text-align/text-align-end-008.html
external/wpt/css/css-text/text-align/text-align-end-016.html
external/wpt/css/css-text/text-align/text-align-end-017.html
external/wpt/css/css-text/text-align/text-align-justify-001.html
external/wpt/css/css-text/text-align/text-align-justify-002.html
external/wpt/css/css-text/text-align/text-align-justify-003.html
external/wpt/css/css-text/text-align/text-align-start-015.html
external/wpt/css/css-text/text-align/text-align-start-016.html
external/wpt/css/css-text/text-align/text-align-start-017.html
external/wpt/css/css-text/white-space/pre-wrap-001.html
external/wpt/css/css-text/white-space/pre-wrap-001.html
external/wpt/css/css-text/white-space/pre-wrap-003.html
external/wpt/css/css-text/white-space/pre-wrap-007.html
external/wpt/css/css-text/white-space/pre-wrap-009.html
external/wpt/css/css-text/white-space/pre-wrap-009.html
external/wpt/css/css-text/white-space/pre-wrap-012.html
external/wpt/css/css-text/white-space/pre-wrap-012.html
external/wpt/css/css-text/white-space/pre-wrap-014.html
external/wpt/css/css-text/white-space/textarea-pre-wrap-003.html
external/wpt/css/css-text/white-space/textarea-pre-wrap-004.html
external/wpt/css/css-text/white-space/textarea-pre-wrap-006.html
external/wpt/css/css-text/white-space/textarea-pre-wrap-011.html

ericwilligers@ can you take a look or triage?
 
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 27 2018

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

commit 4cc5809abac8fa2fc17d275ebc2c8f11c52600a9
Author: Henrik Boström <hbos@chromium.org>
Date: Tue Feb 27 11:49:23 2018

Marked many external/wpt/css/css-text/ tests as flaky.

TBR=ericwilligers@chromium.org

No-Try: True
Bug:  816868 
Change-Id: I0ccb2fa2ad89e2dfcbfb8e15e2dfb8fbc1e8e85b
Reviewed-on: https://chromium-review.googlesource.com/939176
Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539420}
[modify] https://crrev.com/4cc5809abac8fa2fc17d275ebc2c8f11c52600a9/third_party/WebKit/LayoutTests/TestExpectations

Components: -Blink>CSS Blink>Layout
Owner: e...@chromium.org

Comment 4 by hbos@chromium.org, Feb 27 2018

Labels: -Sheriff-Chromium

Comment 5 by pwnall@chromium.org, Feb 28 2018

Also having trouble with:

external/wpt/css/css-text/text-align/text-align-start-007.html
external/wpt/css/css-text/text-align/text-align-start-008.html
external/wpt/css/css-text/text-align/text-align-justify-004.html
external/wpt/css/css-text/text-align/text-align-end-003.html
external/wpt/css/css-text/text-align/text-align-start-001.html
external/wpt/css/css-text/white-space/pre-wrap-005.html
external/wpt/css/css-text/white-space/textarea-pre-wrap-001.html
external/wpt/css/css-text/white-space/textarea-pre-wrap-007.html

https://ci.chromium.org/buildbot/tryserver.chromium.win/win7_chromium_rel_ng/112047
https://ci.chromium.org/buildbot/tryserver.chromium.win/win7_chromium_rel_ng/111676
Project Member

Comment 6 by bugdroid1@chromium.org, Feb 28 2018

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

commit d9aedf21200660473c243bf6d5623bc187496db4
Author: Tsuyoshi Horo <horo@chromium.org>
Date: Wed Feb 28 06:05:08 2018

Mark more css-text WPT tests as flaky

https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=external/wpt/css/css-text/

Bug:  816868 
Change-Id: Ie3d6bc7760df163d873d6b5ddd30a41d3ea6f729
TBR: ericwilligers@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/940702
Reviewed-by: Tsuyoshi Horo <horo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539727}
[modify] https://crrev.com/d9aedf21200660473c243bf6d5623bc187496db4/third_party/WebKit/LayoutTests/TestExpectations

Comment 7 by e...@chromium.org, Mar 1 2018

Cc: kojii@chromium.org e...@chromium.org
Labels: -Pri-1 Pri-2
Owner: ----
Status: Available (was: Assigned)

Comment 8 by kojii@chromium.org, Mar 1 2018

Mergedinto: 817030
Status: Duplicate (was: Available)
I think  issue 817030  has taken care of this.
Project Member

Comment 9 by bugdroid1@chromium.org, Mar 2 2018

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

commit 31b38743b5795ed38237fab77fd3f34892981b50
Author: Manuel Rego Casasnovas <rego@igalia.com>
Date: Fri Mar 02 03:53:26 2018

[css-text] Remove flaky tests from TestExpectations

They were fixed in r539748.

BUG= 816868 , 817030 

Change-Id: I8c9cdf2cefd78ab0ba917e8e78e338a795cbacd1
Reviewed-on: https://chromium-review.googlesource.com/942924
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540440}
[modify] https://crrev.com/31b38743b5795ed38237fab77fd3f34892981b50/third_party/WebKit/LayoutTests/TestExpectations

Sign in to add a comment