New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 675907 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Layout Test fast/input/placeholder-wrongly-placed-if-too-tall.html is flaky

Project Member Reported by yhirano@chromium.org, Dec 20 2016

Issue description

Cc: msten...@opera.com
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 20 2016

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

commit 641a844992f7758585f775696b95aae9f922be75
Author: yhirano <yhirano@chromium.org>
Date: Tue Dec 20 10:45:49 2016

Mark fast/input/placeholder-wrongly-placed-if-too-tall.html as flaky

BUG= 675907 
TBR=jchaffraix@chromium.org, mstensho@opera.com
NOTRY=true

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

[modify] https://crrev.com/641a844992f7758585f775696b95aae9f922be75/third_party/WebKit/LayoutTests/TestExpectations

Components: -Blink>Input Blink>Forms

Comment 4 by tkent@chromium.org, Dec 20 2016

Cc: -jchaffraix@chromium.org
Status: Available (was: Untriaged)
Project Member

Comment 5 by bugdroid1@chromium.org, Jan 13 2017

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

commit 5cc4c469882d169f8efb789ff82235252f40f270
Author: tkent <tkent@chromium.org>
Date: Fri Jan 13 02:37:08 2017

Tentative fix for flakiness of fast/input/placeholder-wrongly-placed-if-too-tall.html.

The test quickly finishes usually, but sometimes it doesn't finish.
We guess it's possible to call checkInputs() before loading js-test.js.

This CL doesn't remove a TestExpectations entry. We'll check if the flakiness is
gone later.

BUG= 675907 

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

[modify] https://crrev.com/5cc4c469882d169f8efb789ff82235252f40f270/third_party/WebKit/LayoutTests/fast/input/placeholder-wrongly-placed-if-too-tall.html

Project Member

Comment 6 by bugdroid1@chromium.org, Jan 15 2017

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

commit 689cad8bd338a99def2adf0ce3b3635064c32292
Author: tkent <tkent@chromium.org>
Date: Sun Jan 15 23:38:28 2017

Remove a flakiness entry for placeholder-wrongly-placed-if-too-tall.html.

The flakiness was gone by r443475.

BUG= 675907 
TBR=yosin@chromium.org
NOTRY=true

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

[modify] https://crrev.com/689cad8bd338a99def2adf0ce3b3635064c32292/third_party/WebKit/LayoutTests/TestExpectations

Comment 7 by tkent@chromium.org, Jan 15 2017

Owner: tkent@chromium.org
Status: Fixed (was: Available)

Sign in to add a comment