New issue
Advanced search Search tips

Issue 674396 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 653722
Owner:
Closed: Mar 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Layout Test compositing/reflections/nested-reflection-transition.html is failing

Project Member Reported by ksakamoto@chromium.org, Dec 15 2016

Issue description

The following layout test is failing on WebKit Win7 (dbg)

compositing/reflections/nested-reflection-transition.html

Probable cause:

started failing since crrev.com/2569063002.

 
Project Member

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

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

commit e9cfdd1cf073da6c905c29d47839b76400e0b08b
Author: ksakamoto <ksakamoto@chromium.org>
Date: Thu Dec 15 04:17:08 2016

Mark compositing/reflections/nested-reflection-transition.html as failing on Win

BUG= 674396 ,653722
TBR=schenney@chromium.org
NOTRY=TRUE

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

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

Owner: ericwilligers@chromium.org
https://codereview.chromium.org/2675913003/ is to fix this.

Project Member

Comment 4 by bugdroid1@chromium.org, Feb 22 2017

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

commit cd543f8a4f13cc05b0c4f9dbf8f1497a8b6b5845
Author: ericwilligers <ericwilligers@chromium.org>
Date: Wed Feb 22 11:15:05 2017

Fix flaky compositing/reflections/nested-reflection-transition.html

We now wait for the compositor before completing the test and
capturing pixels.

BUG= 674396 , 674468 

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

[modify] https://crrev.com/cd543f8a4f13cc05b0c4f9dbf8f1497a8b6b5845/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/cd543f8a4f13cc05b0c4f9dbf8f1497a8b6b5845/third_party/WebKit/LayoutTests/animations/resources/animation-test-helpers.js

Comment 6 by fdoray@chromium.org, Feb 22 2017

 Issue 695070  has been merged into this issue.

Comment 7 by fdoray@chromium.org, Feb 22 2017

webkit_tests failing on 2 builders

Builders failed on: 
- WebKit Linux Trusty Leak: 
  https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak
- WebKit Win7: 
  https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win7

07:37:24.800 1916 [7338/50039] compositing/reflections/nested-reflection-transition.html failed unexpectedly (image diff)
07:37:24.799 4340 worker/6 compositing/reflections/nested-reflection-transition.html failed:
07:37:24.800 4340 worker/6  image diff

Comment 8 by fdoray@chromium.org, Feb 22 2017

Labels: Hotlist-Sheriff-Chromium
Project Member

Comment 9 by bugdroid1@chromium.org, Feb 22 2017

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

commit 2786003858a820e942180760173db7c5d2e61251
Author: fdoray <fdoray@chromium.org>
Date: Wed Feb 22 22:22:36 2017

Expect compositing/reflections/nested-reflection-transition.html to fail.

Multiple bots are failing because of this test:

webkit_tests failing on 4 builders

Builders failed on:
- WebKit Linux Trusty:
  https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty
- WebKit Win10:
  https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10
- WebKit Win7:
  https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win7
- WebKit Win7 (dbg):
  https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win7%20%28dbg%29

BUG= 674396 
TBR=schenney@chromium.org

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

[modify] https://crrev.com/2786003858a820e942180760173db7c5d2e61251/third_party/WebKit/LayoutTests/TestExpectations

Comment 10 by treib@chromium.org, Feb 24 2017

Components: Tests>Disabled
Labels: -Hotlist-Sheriff-Chromium
Taking out of the sheriff queue, since the tests have been disabled and there's an owner assigned.
Mergedinto: 653722
Owner: schenney@chromium.org
Status: Duplicate (was: Assigned)
Components: Tests>Disabled
Labels: Test-Disabled

Sign in to add a comment