Bug 108213 - [Qt] HTML5 video - sound volume bar out of widget
Summary: [Qt] HTML5 video - sound volume bar out of widget
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Allan Sandfeld Jensen
Depends on:
Blocks: QtWebkit23 103747 112146
  Show dependency treegraph
Reported: 2013-01-29 12:32 PST by jingdow
Modified: 2013-03-12 06:47 PDT (History)
11 users (show)

See Also:

Patch (2.82 KB, patch)
2013-01-31 03:11 PST, Allan Sandfeld Jensen
no flags Details | Formatted Diff | Diff
Fixes issue also for media files url's (2.33 KB, patch)
2013-01-31 04:54 PST, David Rosca
no flags Details | Formatted Diff | Diff
Patch (3.58 KB, patch)
2013-02-18 03:03 PST, Allan Sandfeld Jensen
no flags Details | Formatted Diff | Diff
Patch (9.38 KB, patch)
2013-02-18 10:09 PST, Allan Sandfeld Jensen
jturcotte: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description jingdow 2013-01-29 12:32:08 PST
Another regression since qtwebkit 2.2. Sound volume bar is often displaced and out of the video widget. Also, If browser supports directly opening of the media, after video plays to the end it disappears.

Page for testing: http://www.w3schools.com/html/tryit.asp?filename=tryhtml5_video_all

Some screenshots of the issue:

1. http://s7.postimage.org/kzo5oesaz/sbissue1.png
2. http://s1.postimage.org/h5isrnysf/sbissue2.png
3. http://s8.postimage.org/ncvzfjuzn/sbissue3.png

Picture nr. 3 is the situation where the video would disappear after playing to the end.
Comment 1 Allan Sandfeld Jensen 2013-01-30 04:55:13 PST
This is not specific to QtWebKit 2.3. I can reproduce the same issue in trunk with Qt5.
Comment 2 Allan Sandfeld Jensen 2013-01-31 03:11:21 PST
Created attachment 185732 [details]
Comment 3 David Rosca 2013-01-31 04:54:01 PST
Created attachment 185752 [details]
Fixes issue also for media files url's
Comment 4 David Rosca 2013-01-31 04:54:20 PST
The patch doesn't fix the issue on: http://www.w3schools.com/html/movie.mp4
My patch fixes it.
Comment 5 jingdow 2013-01-31 06:03:13 PST
Confirming, David's patch fixed the sound volume bar.
Comment 6 Allan Sandfeld Jensen 2013-02-18 03:03:50 PST
Created attachment 188840 [details]
Comment 7 Allan Sandfeld Jensen 2013-02-18 07:51:00 PST
Comment on attachment 188840 [details]

Untagging for review. The underlying issue seems to be we use the Apple MediaControl shadow DOM, but with our own CSS and set of buttons. 

All these patch attempts does is work around the wrong underlying shadow DOM tree.
Comment 8 Allan Sandfeld Jensen 2013-02-18 10:09:12 PST
Created attachment 188914 [details]

Another attempt, this time following Apple CSS, but keeping our style.
Comment 9 Jocelyn Turcotte 2013-03-08 05:02:10 PST
Comment on attachment 188914 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=188914&action=review

I can't say that those coordinates all makes sense to me, but I tested the patch and it's definitely an improvement.

> Source/WebCore/css/mediaControlsQt.css:39
>      bottom: 0px;

This line is already in mediaControls.css, you should be able to remove it.
Comment 10 Allan Sandfeld Jensen 2013-03-08 06:25:08 PST
Committed r145214: <http://trac.webkit.org/changeset/145214>