1480036 - Allow scaling all MSG volume with a pref

Status: RESOLVED FIXED

(Core :: Audio/Video: MediaStreamGraph, enhancement, P3)

Description

[Paul Adenot (:padenot)]

This is so that it's less annoying when running the tests locally.

Comment 1

[Paul Adenot (:padenot)]

Attachment: Bug 1480036 - Allow scaling all MSG volume with a pref. r?pehrsons

Comment 2

[Paul Adenot (:padenot)]

Attachment: Bug 1480036 - Allow forcing a specific audio output device from a test. r?pehrsons

Comment 3

[Paul Adenot (:padenot)]

Attachment: Bug 1480036 - Stop relying on having the default device set globally. r?pehrsons

Comment 4

[Phabricator Automation]

Comment on attachment 8996756 [details]
Bug 1480036 - Allow forcing a specific audio output device from a test.  r?pehrsons

Andreas Pehrson [:pehrsons] has approved the revision.

https://phabricator.services.mozilla.com/D2609

Comment 5

[Paul Adenot (:padenot)]

I can't seem to be able to bend our test infrastructure to only set the loopback devices when needed. i'm going to adopt the following strategy:
- On Linux, we have loopback devices. Force their use in gecko, via a pref, so that we don't mutate/rely on the machine state. This means we can listen to music on a normal audio output device while running the tests.
- On other platforms, scale the volume. This means that we won't be hearing ear-piercing sine waves.

Comment 6

[Paul Adenot (:padenot)]

Attachment: Bug 1480036 - Allow forcing a specific audio output device from a test. r?pehrsons

Comment 7

[Paul Adenot (:padenot)]

Attachment: Bug 1480036 - Stop relying on having the default device set globally. r?pehrsons

Comment 8

[Andreas Pehrson [:pehrsons]]

Comment on attachment 9001598 [details]
Bug 1480036 - Allow forcing a specific audio output device from a test.  r?pehrsons

Andreas Pehrson [:pehrsons] has approved the revision.

Comment 9

[Andreas Pehrson [:pehrsons]]

Comment on attachment 9001599 [details]
Bug 1480036 - Stop relying on having the default device set globally. r?pehrsons

Andreas Pehrson [:pehrsons] has approved the revision.

Comment 10

[Andreas Pehrson [:pehrsons]]

Comment on attachment 8996652 [details]
Bug 1480036 - Allow scaling all MSG volume with a pref. r?pehrsons

Andreas Pehrson [:pehrsons] has approved the revision.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3f99a091261f
Allow forcing a specific audio output device from a test.  r=pehrsons
https://hg.mozilla.org/integration/mozilla-inbound/rev/352e47aeafac
Stop relying on having the default device set globally. r=pehrsons
https://hg.mozilla.org/integration/mozilla-inbound/rev/f2dc5d8ac9ac
Allow scaling all MSG volume with a pref. r=pehrsons

Comment 12

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/3f99a091261f
https://hg.mozilla.org/mozilla-central/rev/352e47aeafac
https://hg.mozilla.org/mozilla-central/rev/f2dc5d8ac9ac