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

Issue 646050 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

[Sheriff] Build failure

Project Member Reported by lazyboy@chromium.org, Sep 12 2016

Issue description

Build is broken:
webkit_tests http/tests/media/video-buffered-range-contains-currentTime.html
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Fmedia%2Fvideo-buffered-range-contains-currentTime.html&testType=webkit_tests
webkit_tests media/video-zoom-controls.html
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=media%2Fvideo-zoom-controls.html&testType=webkit_tests

Revision range:
chromium 417921 : 417926

Failing builders:
WebKit Win10: https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10

I think this is mlamouri's https://codereview.chromium.org/2314403004

I'm going to disable those two tests.
 
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux/builds/70282

media/video-zoom-controls.html also failing on Linux
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 12 2016

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

commit 1b042808c32e9c1579e5e889e479c7f7a832b223
Author: lazyboy <lazyboy@chromium.org>
Date: Mon Sep 12 20:08:19 2016

[Sheriff] Disable two tests in webkit_tests on Win10.
media/video-zoom-controls.html
http/tests/media/video-buffered-range-contains-currentTime.html

And one of them on Linux.
media/video-zoom-controls.html

Reason: Failed on bots:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/14695/steps/webkit_tests/logs/stdio
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux/builds/70282

Most likely since:
https://codereview.chromium.org/2314403004

TBR=mlamouri@chromium.org
BUG= 646050 

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

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

Project Member

Comment 3 by bugdroid1@chromium.org, Sep 13 2016

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

commit 8cf8d8f908021c6240d33c1db9b661b539144570
Author: mlamouri <mlamouri@chromium.org>
Date: Tue Sep 13 14:22:12 2016

Re-enable LayoutTests after rebaseline.

BUG= 646050 

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

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

Status: Fixed (was: Assigned)
That CL passed CQ so hopefully, there will be no bad surprise :)

Sign in to add a comment