Bug 187138 - [GStreamer] Media elements with maximal volume should be restricted to a configurable value
Summary: [GStreamer] Media elements with maximal volume should be restricted to a conf...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: Other
Hardware: Unspecified Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-06-28 04:18 PDT by kailueke
Modified: 2018-06-28 23:34 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description kailueke 2018-06-28 04:18:31 PDT
Despite the fix in https://bugs.webkit.org/show_bug.cgi?id=118974 still there are situations where the sound volume jumps to maximum. Depending on the audio device this can at least shock the user.

I think pulseaudio flat volumes are a good idea, but WebKit(Gtk) should offer a setting for a maximal volume that is used for elements and also applies to the JS API. This stops propagation of a harmful high system volume. Users can still raise the system volume by hand if needed (pulseaudio then raises the media's volume up by the same rate).

Any feedback on this idea? Disabling flat volumes is not a good option because it often requires to open the audio settings and raise an audio stream's volume because two volume levels just get messed up.
Comment 1 Michael Catanzaro 2018-06-28 05:19:22 PDT
CC Calvaris since he understands this issue, but I'm not sure why the resolution would be different now than it was a few years ago.

Flat volumes are nowadays globally disabled in all major distributions (Ubuntu, Fedora, Arch) except perhaps Debian (not sure about this one) since the PulseAudio developers agreed that was the best way to resolve this issue.
Comment 2 Xabier Rodríguez Calvar 2018-06-28 23:34:39 PDT
There's already a set volume API for the page. That won't set a high volume but should scale down the volume that is already there.

We don't contemplate changing this because (I already explained it many times) we don't want to break any integration with GNOME/Pulse volumes. It's a known issue yes and as you say, caused by flat volumes.