1519350 - Audio volume unexpectedly is changed to 100% on video when seek

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect, P2)

Description

[Alice0775 White]

[Tracking Requested - why for this release]: volume broken after seek time

Reproducible: always:

Steps to reproduce:

1. open youtube video and playback
   (e.g https://www.youtube.com/watch?v=QXimzCqh3d0)
2. Change volume to very small such as 2%
3. Seek the time slider

Actual Results:
The volume slider position is correct.
However, Suddenly, the volume is raised to 100%.

Expected Results:
Volume should not be changed

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=876faba9c48f5bee522dae19d91acde4b60efc86&tochange=849f81f21979e1eceec29e2b3f7dd769d64496e4

Regressed by:
849f81f21979 Alex Chronopoulos — Bug 1515549 - Avoid resetting volume in every AudioSink restart in case volume has changed outside firefox. r=jya

@Alex,
Your patch seems to cause the problem, Can you please look into this?

Comment 1

[Alice0775 White]

Attachment: about:support

Comment 2

[Alice0775 White]

Attachment: video codecs, etc.

Comment 3

[Alice0775 White]

Affected to any video.
https://assets.mozilla.net/video/firefox-home/quantum-vs-chrome.webm

Comment 4

[Alex Chronopoulos [:achronop]]

Thanks Alice, I'll take a look. BTW it does not happen on Linux and OSX here. Can you verify that?

Comment 5

[Alice0775 White]

(In reply to Alex Chronopoulos [:achronop] from comment #4)

Thanks Alice, I'll take a look. BTW it does not happen on Linux and OSX
here. Can you verify that?

At least, I can not reproduce this problem on Ubuntu18.04.

Comment 6

[Alice0775 White]

Fixed by backed out the offending patch(Bug 1515549).