Skip to content

Conversation

@servo-wpt-sync
Copy link
Collaborator

@servo-wpt-sync servo-wpt-sync commented Nov 20, 2025

Following the HTML specification, on the volume or mute attrubute changes the user agent must queue a media element task given the media element to fire an event named volumechange at the media element (the generation_id value will be captured to allow cancel this task - media-element-load-algorithm steps 3-5)

See https://html.spec.whatwg.org/multipage/#event-media-volumechange
See https://html.spec.whatwg.org/multipage/#media-element-load-algorithm

Changed the naming of the playbackRate and defaultPlaybackRate members to match Rust naming conventions (snake_case).

Testing: Added the cancellable subtest to event_volumechange test

  • html/semantics/embedded-content/media-elements/event_volumechange.html

Reviewed in servo/servo#40763

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Servo project.

Following the HTML specification, on the `volume` or `mute`
attrubute changes the user agent must queue a media element
task given the media element to fire an event named
`volumechange` at the media element (the `generation_id`
value will be captured to allow cancel this task -
`media-element-load-algorithm` steps 3-5)

See https://html.spec.whatwg.org/multipage/#event-media-volumechange
See https://html.spec.whatwg.org/multipage/#media-element-load-algorithm

Changed the naming of the `playbackRate` and `defaultPlaybackRate`
members to match Rust naming conventions (`snake_case`).

Testing: Added the `cancellable` subtest to `event_volumechange` test
- html/semantics/embedded-content/media-elements/event_volumechange.html

Signed-off-by: Andrei Volykhin <andrei.volykhin@gmail.com>
@servo-wpt-sync servo-wpt-sync merged commit 5408908 into web-platform-tests:master Nov 20, 2025
17 checks passed
@servo-wpt-sync servo-wpt-sync deleted the servo_export_40763 branch November 20, 2025 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants