Skip to content

Commit

Permalink
Bug 1862098 [wpt PR 42856] - Prepare to enable HDR media capabilities…
Browse files Browse the repository at this point in the history
… by default., a=testonly

Automatic update from web-platform-tests
Prepare to enable HDR media capabilities by default.

I2S approved here:
https://groups.google.com/a/chromium.org/g/blink-dev/c/0neM-5GDn8I/m/ymH32sDDAgAJ

This cleans up an unnecessary feature flag and implements
the rest of the feature per the current spec. Adds a test
for mismatched color space information.

w3c/media-capabilities#152 covers
how we should reject mismatched color parameters, though a
PR to add it to the spec hasn't yet been made. Safari has
implemented this behavior though, so copy it.

Bug: 1048045
Change-Id: I4fee62361c697922bb041724f4e4ac8c8e9d5c2d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4989038
Reviewed-by: Johannes Kron <kronchromium.org>
Commit-Queue: Dale Curtis <dalecurtischromium.org>
Cr-Commit-Position: refs/heads/main{#1217958}

--

wpt-commits: f1f1925c4876c73f932d35e0a038b619b861ed83
wpt-pr: 42856

UltraBlame original commit: 04c64b3491fa73115ddc56ccc442cb7655e7bd2e
  • Loading branch information
marco-c committed Nov 21, 2023
1 parent 5802177 commit fdd62bf
Showing 1 changed file with 150 additions and 12 deletions.
162 changes: 150 additions & 12 deletions testing/web-platform/tests/media-capabilities/decodingInfo.any.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ use
strict
'
;
var
const
minimalVideoConfiguration
=
{
Expand Down Expand Up @@ -40,7 +40,7 @@ framerate
24
}
;
var
const
minimalAudioConfiguration
=
{
Expand All @@ -59,7 +59,7 @@ opus
'
}
;
var
const
audioConfigurationWithSpatialRendering
=
{
Expand All @@ -81,7 +81,7 @@ spatialRendering
true
}
;
var
const
videoConfigurationWithDynamicRange
=
{
Expand All @@ -102,6 +102,16 @@ vp09
10
.
08
.
00
.
09
.
16
.
09
.
00
"
'
width
Expand All @@ -118,20 +128,21 @@ framerate
24
hdrMetadataType
:
"
'
smpteSt2086
"
'
colorGamut
:
"
srgb
"
'
rec2020
'
transferFunction
:
"
srgb
"
'
pq
'
}
;
promise_test
(
t
Expand Down Expand Up @@ -2346,6 +2357,133 @@ t
=
>
{
let
bt709Config
=
videoConfigurationWithDynamicRange
;
bt709Config
.
contentType
=
'
video
/
webm
;
codecs
=
"
vp09
.
00
.
10
.
08
"
'
;
return
navigator
.
mediaCapabilities
.
decodingInfo
(
{
type
:
'
file
'
video
:
bt709Config
}
)
.
then
(
ability
=
>
{
assert_equals
(
typeof
ability
.
supported
'
boolean
'
)
;
assert_equals
(
typeof
ability
.
smooth
'
boolean
'
)
;
assert_equals
(
typeof
ability
.
powerEfficient
'
boolean
'
)
;
assert_equals
(
typeof
ability
.
keySystemAccess
'
object
'
)
;
assert_false
(
ability
.
supported
)
;
}
)
;
}
'
Test
that
decodingInfo
with
mismatched
codec
color
space
is
unsupported
'
)
;
promise_test
(
t
=
>
{
return
promise_rejects_js
(
Expand Down

0 comments on commit fdd62bf

Please sign in to comment.