Skip to content

Commit

Permalink
REGRESSION (256665@main): [ Ventura ] fast/images/avif-as-image.html …
Browse files Browse the repository at this point in the history
…is a consistent image failure

https://bugs.webkit.org/show_bug.cgi?id=248544
rdar://102823196

Reviewed by Simon Fraser.

libavif does not render fast/images/resources/green-313x313.avif and
fast/images/resources/green-400x400.avif correctly.

See AOMediaCodec/av1-avif#195.

We need to fix the expectation file fast/images/avif-as-image-expected.html
with the correct rendering; i.e. the image should be lime not green. This will
require marking this test [ ImageOnlyFailure ] for non Apple ports and macOS
down-level ports.

* LayoutTests/TestExpectations:
* LayoutTests/fast/images/avif-as-image-expected.html:
* LayoutTests/platform/ios/TestExpectations:
* LayoutTests/platform/mac-ventura/TestExpectations:
* LayoutTests/platform/mac/TestExpectations:

Canonical link: https://commits.webkit.org/257738@main
  • Loading branch information
local authored and Said Abou-Hallawa committed Dec 12, 2022
1 parent 2d2f8ac commit c7d2aef
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 6 deletions.
3 changes: 2 additions & 1 deletion LayoutTests/TestExpectations
Original file line number Diff line number Diff line change
Expand Up @@ -2252,8 +2252,9 @@ fast/images/animated-webp.html [ Skip ]
# These tests don't have to be platform-specific, but they are only implemented on Mac now.
fast/images/eps-as-image.html [ Skip ]

# libavif does not render resources/green-400x400.avif correctly.
# libavif does not render resources/green-313x313.avif and resources/green-400x400.avif correctly.
# See https://github.com/AOMediaCodec/av1-avif/issues/195
fast/images/avif-as-image.html [ ImageOnlyFailure ]
fast/images/avif-heif-container-as-image.html [ ImageOnlyFailure ]

# Only applicable on platforms with JPEG XL support
Expand Down
2 changes: 1 addition & 1 deletion LayoutTests/fast/images/avif-as-image-expected.html
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
div {
width: 313px;
height: 313px;
background-color: green;
background-color: lime;
}
</style>
<body>
Expand Down
1 change: 0 additions & 1 deletion LayoutTests/platform/ios/TestExpectations
Original file line number Diff line number Diff line change
Expand Up @@ -3787,7 +3787,6 @@ imported/w3c/web-platform-tests/css/selectors/selection-image-002.html [ ImageOn

# AVIF tests
webkit.org/b/247831 fast/images/avif-image-decoding.html [ Failure ]
webkit.org/b/247831 fast/images/avif-as-image.html [ ImageOnlyFailure ]
webkit.org/b/247831 fast/images/animated-avif.html [ Timeout ]

# Screen Orientation API doesn't support nested frames yet
Expand Down
2 changes: 0 additions & 2 deletions LayoutTests/platform/mac-ventura/TestExpectations
Original file line number Diff line number Diff line change
@@ -1,3 +1 @@
webkit.org/b/245686 [ Ventura+ ] fast/text/system-font-fallback.html [ ImageOnlyFailure ]

webkit.org/b/248544 [ Ventura+ ] fast/images/avif-as-image.html [ ImageOnlyFailure ]
2 changes: 1 addition & 1 deletion LayoutTests/platform/mac/TestExpectations
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,7 @@ fast/images/heic-as-background-image.html [ Pass ]
fast/images/mac/play-all-pause-all-animations-context-menu-items.html [ Skip ]

# AVIF tests
[ Ventura+ ] fast/images/avif-as-image.html [ Pass ]
[ Ventura+ ] fast/images/avif-heif-container-as-image.html [ Pass ]

# <rdar://problem/5647952> fast/events/mouseout-on-window.html needs mac DRT to issue mouse out events
Expand Down Expand Up @@ -2400,7 +2401,6 @@ svg/transforms/translation-tiny-element.svg [ ImageOnlyFailure ]
[ Ventura+ x86_64 ] http/tests/media/now-playing-info.html [ Pass Failure ]
[ BigSur+ x86_64 ] animations/stop-animation-on-suspend.html [ Pass Failure ]
[ Ventura+ ] imported/w3c/web-platform-tests/mathml/relations/css-styling/display-2.html [ Failure ]
[ Ventura+ ] fast/images/avif-as-image.html [ ImageOnlyFailure Crash ]
[ BigSur+ ] fast/images/animated-avif.html [ Pass ImageOnlyFailure Timeout Crash ]
[ Ventura+ ] animations/animation-events-not-cancelable.html [ Pass Failure ]
[ Bigsur+ ] animations/remove-syncing-animation.html [ Pass Failure ]
Expand Down

0 comments on commit c7d2aef

Please sign in to comment.