WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
219897
REGRESSION (
r270806
): [iOS] imported/w3c/web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-distance-clamping.html is frequently failing
https://bugs.webkit.org/show_bug.cgi?id=219897
Summary
REGRESSION (r270806): [iOS] imported/w3c/web-platform-tests/webaudio/the-audi...
Ryan Haddad
Reported
2020-12-15 09:52:41 PST
imported/w3c/web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-distance-clamping.html has started failing frequently on iOS simulator release bots --- /Volumes/Data/slave/ios-simulator-14-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-distance-clamping-expected.txt +++ /Volumes/Data/slave/ios-simulator-14-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-distance-clamping-actual.txt @@ -24,16 +24,16 @@ PASS > [min-distance] PASS Model: linear: Distance (0.01) is outside the range [1, 10000] is equal to true. PASS Test panner output {"distance":0.01,"distanceModel":"linear"} is identical to the array [expected array]. -PASS Model: exponential: Distance (0.01) is outside the range [1, 10000] is equal to true. -PASS Test panner output {"distance":0.01,"distanceModel":"exponential"} is identical to the array [expected array]. -PASS Model: inverse: Distance (0.01) is outside the range [1, 10000] is equal to true. -PASS Test panner output {"distance":0.01,"distanceModel":"inverse"} is identical to the array [expected array]. -PASS Model: linear: Distance (2) is outside the range [10, 1000] is equal to true. -PASS Test panner output {"distance":2,"distanceModel":"linear","maxDistance":1000,"refDistance":10} is identical to the array [expected array]. PASS Model: exponential: Distance (2) is outside the range [10, 1000] is equal to true. PASS Test panner output {"distance":2,"distanceModel":"exponential","maxDistance":1000,"refDistance":10} is identical to the array [expected array]. PASS Model: inverse: Distance (2) is outside the range [10, 1000] is equal to true. PASS Test panner output {"distance":2,"distanceModel":"inverse","maxDistance":1000,"refDistance":10} is identical to the array [expected array]. +PASS Model: linear: Distance (2) is outside the range [10, 1000] is equal to true. +PASS Test panner output {"distance":2,"distanceModel":"linear","maxDistance":1000,"refDistance":10} is identical to the array [expected array]. +PASS Model: inverse: Distance (0.01) is outside the range [1, 10000] is equal to true. +PASS Test panner output {"distance":0.01,"distanceModel":"inverse"} is identical to the array [expected array]. +PASS Model: exponential: Distance (0.01) is outside the range [1, 10000] is equal to true. +PASS Test panner output {"distance":0.01,"distanceModel":"exponential"} is identical to the array [expected array]. PASS < [min-distance] All assertions passed. (total 12 assertions) PASS > [max-distance] PASS Model: linear: Distance (20000) is outside the range [1, 10000] is equal to true. @@ -42,10 +42,10 @@ PASS Test panner output {"distance":21000,"distanceModel":"exponential"} is identical to the array [expected array]. PASS Model: inverse: Distance (23000) is outside the range [1, 10000] is equal to true. PASS Test panner output {"distance":23000,"distanceModel":"inverse"} is identical to the array [expected array]. +PASS Model: exponential: Distance (5000) is outside the range [10, 1000] is equal to true. +PASS Test panner output {"distance":5000,"distanceModel":"exponential","maxDistance":1000,"refDistance":10} is identical to the array [expected array]. PASS Model: linear: Distance (5000) is outside the range [10, 1000] is equal to true. PASS Test panner output {"distance":5000,"distanceModel":"linear","maxDistance":1000,"refDistance":10} is identical to the array [expected array]. -PASS Model: exponential: Distance (5000) is outside the range [10, 1000] is equal to true. -PASS Test panner output {"distance":5000,"distanceModel":"exponential","maxDistance":1000,"refDistance":10} is identical to the array [expected array]. PASS Model: inverse: Distance (5000) is outside the range [10, 1000] is equal to true. PASS Test panner output {"distance":5000,"distanceModel":"inverse","maxDistance":1000,"refDistance":10} is identical to the array [expected array]. PASS < [max-distance] All assertions passed. (total 12 assertions)
https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fwebaudio%2Fthe-audio-api%2Fthe-pannernode-interface%2Fpanner-distance-clamping.html
Attachments
Patch
(6.22 KB, patch)
2020-12-16 08:38 PST
,
Chris Dumez
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2020-12-15 09:52:59 PST
<
rdar://problem/72346130
>
Ryan Haddad
Comment 2
2020-12-15 09:53:54 PST
The first failure on iOS appears at
r270808
. Maybe
r270806
is related?
Ryan Haddad
Comment 3
2020-12-15 15:42:55 PST
I have confirmed the regression point is
r270806
. The test doesn't fail when run with
r270805
. run-webkit-tests --ios-simulator imported/w3c/web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-distance-clamping.html --no-retry --iterations 10 --force
Ryan Haddad
Comment 4
2020-12-15 15:45:10 PST
This only appears to be failing with release builds according to test history. Debug test runs are all green.
Ryan Haddad
Comment 5
2020-12-16 07:53:08 PST
Chris, would you please take a look to see if it is an easy fix or if we need to revert the change?
Chris Dumez
Comment 6
2020-12-16 08:23:54 PST
(In reply to Ryan Haddad from
comment #5
)
> Chris, would you please take a look to see if it is an easy fix or if we > need to revert the change?
I will take a look now. This is just an issue with the test though, not a true regression from a change. This is a typical test doing a bunch of async operations in parallel and the order in which they complete is undetermined.
Chris Dumez
Comment 7
2020-12-16 08:38:26 PST
Created
attachment 416339
[details]
Patch
EWS Watchlist
Comment 8
2020-12-16 08:39:17 PST
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see
https://trac.webkit.org/wiki/WPTExportProcess
EWS
Comment 9
2020-12-16 12:42:54 PST
Committed
r270903
: <
https://trac.webkit.org/changeset/270903
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 416339
[details]
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug