[Webkit-unassigned] [Bug 84378] [GTK] media/track/track-cue-rendering-snap-to-lines-not-set.html fails
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Apr 20 16:10:06 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=84378
--- Comment #2 from Philippe Normand <pnormand at igalia.com> 2012-04-20 16:10:06 PST ---
(In reply to comment #1)
> (In reply to comment #0)
> > Consistently. Diff:
> >
> > --- /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/media/track/track-cue-rendering-snap-to-lines-not-set-expected.txt
> > +++ /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/media/track/track-cue-rendering-snap-to-lines-not-set-actual.txt
> > @@ -2,7 +2,7 @@
> > EVENT(canplaythrough)
> >
> > The track container and the media controls should completely cover the video element
> > -EXPECTED (cueDisplayContainer.offsetHeight + controlsPanelElement.offsetHeight == video.videoHeight == 'true') OK
> > +EXPECTED (cueDisplayContainer.offsetHeight + controlsPanelElement.offsetHeight == video.videoHeight == 'true'), OBSERVED 'false' FAIL
> > EXPECTED (cueDisplayContainer.offsetWidth == video.videoWidth == 'true') OK
> >
> > ** Default **
> >
> > This test was introduced in
> > http://trac.webkit.org/changeset/114640
> I believe this particular check fails because the -webkit-media-controls-panel element is absolutely positioned in mediaControlsGTK.
>
> This change from absolute to relative positioning is required because the area on top of the video is now a flexible box, with two main boxes on top of each other: cue container box and control panel element.
>
> If the panel element is absolutely positioned the flexible box is not positioning it, and the cue container expands towards the bottom of the video.
Thanks Victor!
I was just trying to figure out why the media controls panel was invisible now... And I remembered this bug. Will try your proposed solution and upload a patch.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list