New issue
Advanced search Search tips

Issue 921897 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 921835
Owner:
Closed: Jan 16
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

webkit_unit_tests failing on chromium.linux/Linux Tests (dbg)(1)(32)

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Jan 15

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of loyso@google.com

webkit_unit_tests failing on chromium.linux/Linux Tests (dbg)(1)(32)

WorkletAnimationTest.CurrentTimeFromDocumentTimelineIsOffsetByStartTime
fails. Disable it.

Builders failed on: 
- Linux Tests (dbg)(1)(32): 
  https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20Tests%20%28dbg%29%281%29%2832%29


 
Owner: flackr@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 15

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

commit 26045cc0fa599ff21768bb37ab9de9237ccb4a0d
Author: Alexey Baskakov <loyso@chromium.org>
Date: Tue Jan 15 05:42:27 2019

Disable WorkletAnimationTest test.

TBR=flackr@chromium.org

Bug:921897

Change-Id: I3fb513749a0bee88ec52c1fe16be8e4058ef5f6b
Reviewed-on: https://chromium-review.googlesource.com/c/1410773
Commit-Queue: Alexey Baskakov <loyso@chromium.org>
Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622750}
[modify] https://crrev.com/26045cc0fa599ff21768bb37ab9de9237ccb4a0d/cc/animation/worklet_animation_unittest.cc

Comment 3 by fergal@chromium.org, Jan 16 (6 days ago)

Can we mark this test as flaky please? I just ran into it.

Comment 4 by majidvp@google.com, Jan 16 (6 days ago)

Mergedinto: 921835
Status: Duplicate (was: Assigned)

Comment 5 by majidvp@google.com, Jan 16 (6 days ago)

I believe #2 disabled a different (and incorrect) test.

Comment #1 suggests webkit_unit_tests was flaky but the disabled test is a test with the same name in cc. The test have the same exact names (one is for blink side and one is for cc side) which explain the issue.

I am fixing the original issue with a patch soon.

Sign in to add a comment