Bug 215335

Summary: [ macOS wk1 Release ] webanimations/css-transition-retargeting-during-ready-promise.html is a flaky failure
Product: WebKit Reporter: Hector Lopez <hector_i_lopez>
Component: New BugsAssignee: Karl Rackler <rackler>
Status: REOPENED ---    
Severity: Normal CC: dino, ehutchison, graouts, rackler, tsavell, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Description Hector Lopez 2020-08-10 13:25:32 PDT
webanimations/css-transition-retargeting-during-ready-promise.html

This test is a flaky failure on macOS wk1 Release according to history. The earliest occurrence of failure that could be found is at r263493.

History:
https://results.webkit.org/?suite=layout-tests&test=webanimations%2Fcss-transition-retargeting-during-ready-promise.html&limit=50000&platform=mac&style=release&flavor=wk1

Diff:
--- /Volumes/Data/worker/goldengate-applesilicon-production-wk2-tests/build/OpenSource/layout-test-results/webanimations/css-transition-retargeting-during-ready-promise-expected.txt
+++ /Volumes/Data/worker/goldengate-applesilicon-production-wk2-tests/build/OpenSource/layout-test-results/webanimations/css-transition-retargeting-during-ready-promise-actual.txt
@@ -1,3 +1,3 @@
 
-PASS A CSS transition that is retargeted during its ready promise should not use its from style as its before-change style. 
+FAIL A CSS transition that is retargeted during its ready promise should not use its from style as its before-change style. assert_not_equals: The new transition used a different from value. got disallowed value "0px"
Comment 1 Radar WebKit Bug Importer 2020-08-10 13:25:59 PDT
<rdar://problem/66796610>
Comment 2 Hector Lopez 2020-08-10 14:21:42 PDT
Test expectation while investigated:

https://trac.webkit.org/changeset/265449/webkit
Comment 3 Eric Hutchison 2021-06-29 14:35:08 PDT
Issue is now found on Big Sur wk2 as well as wk1 starting at r278452.
Comment 4 Eric Hutchison 2021-06-29 14:58:55 PDT
Created attachment 432536 [details]
Patch
Comment 5 Eric Hutchison 2021-06-29 15:08:26 PDT
Created attachment 432538 [details]
Patch
Comment 6 Eric Hutchison 2021-06-29 15:15:33 PDT
Created attachment 432539 [details]
Patch
Comment 7 Truitt Savell 2021-06-29 15:17:59 PDT
Comment on attachment 432539 [details]
Patch

Clearing flags on attachment: 432539

Committed r279390 (239255@main): <https://commits.webkit.org/239255@main>
Comment 8 Karl Rackler 2022-04-21 09:23:25 PDT
[ Monterey Release ] webanimations/css-transition-retargeting-during-ready-promise.html is a flaky failure

Description:
webanimations/css-transition-retargeting-during-ready-promise.html

The first failure that I saw on the dashboard was 3/17/2022 at r291466, which deals with smaller checkboxes that no longer vertically align with related text,  but this does not appear relevant to causing this issue.

History:
https://results.webkit.org/?suite=layout-tests&test=webanimations%2Fcss-transition-retargeting-during-ready-promise.html&platform=mac&limit=50000

Diff:
--- /Volumes/Data/worker/monterey-release-applesilicon-tests-wk2/build/layout-test-results/webanimations/css-transition-retargeting-during-ready-promise-expected.txt
+++ /Volumes/Data/worker/monterey-release-applesilicon-tests-wk2/build/layout-test-results/webanimations/css-transition-retargeting-during-ready-promise-actual.txt
@@ -1,3 +1,3 @@
 
-PASS A CSS transition that is retargeted during its ready promise should not use its from style as its before-change style.
+FAIL A CSS transition that is retargeted during its ready promise should not use its from style as its before-change style. assert_not_equals: The new transition used a different from value. got disallowed value "0px"


I have marked this test as a flaky failure while this issue is investigated.
Comment 9 Karl Rackler 2022-04-21 09:25:28 PDT
Pull request: https://github.com/WebKit/WebKit/pull/345
Comment 10 EWS 2022-04-21 09:28:55 PDT
Committed r293171 (249853@main): <https://commits.webkit.org/249853@main>

Reviewed commits have been landed. Closing PR #345 and removing active labels.
Comment 11 Karl Rackler 2022-04-21 09:29:14 PDT
Reopened because the issue is still being investigated.