Bug 145639 – [GTK] Volume bar is broken

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Xabier Rodríguez Calvar]

Created attachment 254259 [details]
Wrong

Currently it looks like the attached screenshot and it is completely non-funcional.

I suspect something might be removed from the RenderTheme* that removed the only part that were handled in C++, like the volume path and the way it was filed up to the volume level. If it were like that, funny that nobody asked me...

[Xabier Rodríguez Calvar]

Created attachment 254260 [details]
Right

You see here the volume path inside the volume bar, it is filled up to its volume level and the thumb is smaller.

[Michael Catanzaro]

Works in 2.8, so this must have broken between February and June.

[ChangSeok Oh]

I'm investigating this. Let me take over it.

[ChangSeok Oh]

Created attachment 260855 [details]
Patch

[Xabier Rodríguez Calvar]

(In reply to comment #4)
> Created attachment 260855 [details]
> Patch

Good! This does the trick.

Have you run the tests, including pixel and maybe remove any possible failing expectations?

Carlos, this patch has to enter the next release, otherwise volume bar can be crippled.

[ChangSeok Oh]

(In reply to comment #5)
> (In reply to comment #4)
> > Created attachment 260855 [details]
> > Patch
> 
> Good! This does the trick.
> 
> Have you run the tests, including pixel and maybe remove any possible
> failing expectations?
I have not. Let me see if there is something that I can do.
BTW did we block some tests due to this issue before?

[Xabier Rodríguez Calvar]

(In reply to comment #6)
> BTW did we block some tests due to this issue before?

I haven't checked either, but there were some tests (with pixel) checking the volume bar so I bet that there must be something in the expectations or we just didn't care and it was failing so far.

[ChangSeok Oh]

Created attachment 260909 [details]
Patch

[ChangSeok Oh]

media/click-volume-bar-not-pausing.html
media/video-volume-slider.html
media/volume-bar-empty-when-muted.html

These seem relevant.
media/click-volume-bar-not-pausing.html and media/volume-bar-empty-when-muted.html are unblocked here. But for media/video-volume-slider.html, it looks more touch required. It is flacky. It passes manually, but not with TestRunner.
I think a timing issue involved here. So I would like to handle it in another ticket by modifying the test.

[ChangSeok Oh]

*** Bug 142491 has been marked as a duplicate of this bug. ***

[ChangSeok Oh]

*** Bug 132253 has been marked as a duplicate of this bug. ***

[ChangSeok Oh]

https://bugs.webkit.org/show_bug.cgi?id=149032

[Xabier Rodríguez Calvar]

LGTM

[Philippe Normand]

Comment on attachment 260909 [details]
Patch

Thanks for fixing this ChangSeok!

[ChangSeok Oh]

Comment on attachment 260909 [details]
Patch

@philn My pleasure =)

[WebKit Commit Bot]

Comment on attachment 260909 [details]
Patch

Clearing flags on attachment: 260909

Committed r189566: <http://trac.webkit.org/changeset/189566>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Michael Catanzaro]

Reopening because test media/volume-bar-empty-when-muted.html has been flaky since this patch was committed. Previously the test always failed; now it almost always fails, but sometimes passes.

[Michael Catanzaro]

Updated expectations in r200936.

[ChangSeok Oh]

> Reopening because test media/volume-bar-empty-when-muted.html has been flaky
> since this patch was committed. Previously the test always failed; now it
> almost always fails, but sometimes passes.

Thanks for the report, Michael. Unfortunately, I have no available linux machine for now so I would not be able to investigate this (as well as other bugs assigned to me) until having a new machine. (probably until the next month) If anyone is interested in this bug, please feel free to take it over from me. If not, I will look into this once again.

[Michael Catanzaro]

> so I would not be able to investigate this (as well as other
> bugs assigned to me) until having a new machine.

Oh that's fine, I wasn't really expecting you to... just keeping track of failing tests.

FWIW, to overload this bug further: I notice the volume bar is draggable, that's not good.

(In reply to comment #20)
> Unfortunately, I have no available linux
> machine for now 

Oh no. Oh no oh no oh no oh no oh no. I hope you survive the month. :(

[Michael Catanzaro]

(In reply to comment #21) 
> FWIW, to overload this bug further: I notice the volume bar is draggable,
> that's not good.

Nevermind, it doesn't seem to be a problem anymore. I noticed that in the ancient WebKit shipped by Debian stable, you can drag the entire volume bar (not just the slider) around the entire web view unless you carefully click exactly on the slider.

[Diego Pino]

This test(s) has been consistenly passing in the last 4000 revisions. Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/70995708>