Bug 222117 – REGRESSION (r273004): [ macOS ] media/modern-media-controls/volume-support/volume-support-drag.html is a flakey text failure

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Robert Jenner]

media/modern-media-controls/volume-support/volume-support-drag.html

is a flakey text failure in macOS. 

HISTORY URL:
https://results.webkit.org/?suite=layout-tests&test=media/modern-media-controls/volume-support/volume-support-drag.html

DIFF URL:
https://build.webkit.org/results/Apple-BigSur-Release-WK2-Tests/r273075%20(310)/media/modern-media-controls/volume-support/volume-support-drag-pretty-diff.html

DIFF TEXT:
@@ -6,19 +6,15 @@
 By default, the volume slider should be set to 0 because the media is muted
 PASS mediaController.controls.volumeSlider.value is 0
 PASS muteButton.getBoundingClientRect().width became different from 0
-PASS volumeSlider.getBoundingClientRect().width became different from 0
-
-Before starting to drag the volume slider, the media should be muted still
-PASS media.muted is true
-
-We initiated a volume slider drag, the media should no longer be muted and the volume set
-PASS media.muted became false
-PASS media.volume became 0.51
-
-We finished dragging to set the volume to 1
-PASS media.volume became 1
-PASS mediaController.controls.volumeSlider.parent.visible is true
 PASS successfullyParsed is true
 
 TEST COMPLETE
 
+
+
+
+
+0:00
+
+0:06
+


Have not reproduced yet. However, history points to flakey failures starting at r273004. There do appear to be related changes at that revision.

[Radar WebKit Bug Importer]

<rdar://problem/74484639>

[Robert Jenner]

Reproduced failure at tip of tree, and by reverting checkout to r273004, and running test: 

run-webkit-test media/modern-media-controls/volume-support/volume-support-drag.html --iterations 100 -f

Ran same test at r273003 checkout, and the failure was not reproducible.

[Robert Jenner]

media/modern-media-controls/tracks-support/mac/tracks-support-show-panel-fullscreen.html

Is a similar test that is failing at the same regression point.

[Robert Jenner]

Created attachment 420879 [details]
Update TestExpectations

[Truitt Savell]

Comment on attachment 420879 [details]
Update TestExpectations

Clearing flags on attachment: 420879

Committed r273105 (234303@main): <https://commits.webkit.org/234303@main>

[Devin Rousso]

Created attachment 421364 [details]
Patch

[Devin Rousso]

Created attachment 421373 [details]
Patch

[EWS]

/Volumes/Data/worker/Commit-Queue/build/LayoutTests/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).

[Devin Rousso]

Created attachment 421375 [details]
Patch

[EWS]

Committed r273366: <https://commits.webkit.org/r273366>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 421375 [details].