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...
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.
Works in 2.8, so this must have broken between February and June.
I'm investigating this. Let me take over it.
Created attachment 260855 [details] Patch
(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.
(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?
(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.
Created attachment 260909 [details] Patch
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.
*** Bug 142491 has been marked as a duplicate of this bug. ***
*** Bug 132253 has been marked as a duplicate of this bug. ***
https://bugs.webkit.org/show_bug.cgi?id=149032
LGTM
Comment on attachment 260909 [details] Patch Thanks for fixing this ChangSeok!
Comment on attachment 260909 [details] Patch @philn My pleasure =)
Comment on attachment 260909 [details] Patch Clearing flags on attachment: 260909 Committed r189566: <http://trac.webkit.org/changeset/189566>
All reviewed patches have been landed. Closing bug.
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.
Updated expectations in r200936.
> 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.
> 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. :(
(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.
This test(s) has been consistenly passing in the last 4000 revisions. Closing bug.
<rdar://problem/70995708>