Merge lp://qastaging/~xavi-garcia-mena/indicator-sound/lp-1478506-use-role-volume into lp://qastaging/indicator-sound/15.10

Proposed by Xavi Garcia
Status: Needs review
Proposed branch: lp://qastaging/~xavi-garcia-mena/indicator-sound/lp-1478506-use-role-volume
Merge into: lp://qastaging/indicator-sound/15.10
Diff against target: 95 lines (+45/-11)
1 file modified
src/volume-control-pulse.vala (+45/-11)
To merge this branch: bzr merge lp://qastaging/~xavi-garcia-mena/indicator-sound/lp-1478506-use-role-volume
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Charles Kerr Pending
Review via email: mp+267985@code.qastaging.launchpad.net

Commit message

Adding changes to use the current role volume instead of getting it from pulse.

Description of the change

Adding changes to use the current role volume instead of getting it from pulse.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks for working on that, I'm just curious but what's the issue with getting it from pulse? Is pulse giving a wrong information? If so why?

Revision history for this message
Xavi Garcia (xavi-garcia-mena) wrote :

Hi Sebastien,

At that point, and only for the "Maroon in trouble" game, I get 100% volume even it is using the multimedia role and that we had set a different volume for that role previously.
For other apps the volume obtained is correct.

I don't know if it has relation with the fact that the sounds played are very short...

I've opened a bug to see if we can get a response.

https://bugs.launchpad.net/ubuntu/+source/pulseaudio/+bug/1485522

Unmerged revisions

501. By Xavi Garcia

Use the current role volume when addng a sink

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches