WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
86328
REGRESSION (
r116900
): media/video-controls-rendering-toggle-display-none.html, media/video-controls-toggling.html failing on GTK Linux 64-bit Release
https://bugs.webkit.org/show_bug.cgi?id=86328
Summary
REGRESSION (r116900): media/video-controls-rendering-toggle-display-none.html...
Philippe Normand
Reported
2012-05-13 18:43:30 PDT
media/video-controls-rendering-toggle-display-none.html and media/video-controls-toggling.html started failing on GTK Linux 64-bit Release in
r116900
<
http://trac.webkit.org/changeset/116900
>.
http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release/r116899%20(23630)/results.html
passed
http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release/r116900%20(23631)/results.html
failed Victor, I guess the GTK mediaControls CSS needs another update? I'll flag those 2 tests for now.
Attachments
Add attachment
proposed patch, testcase, etc.
Victor Carbune
Comment 1
2012-05-14 05:05:08 PDT
(In reply to
comment #0
)
> media/video-controls-rendering-toggle-display-none.html and media/video-controls-toggling.html started failing on GTK Linux 64-bit Release in
r116900
<
http://trac.webkit.org/changeset/116900
>. > >
http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release/r116899%20(23630)/results.html
passed >
http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release/r116900%20(23631)/results.html
failed > > Victor, I guess the GTK mediaControls CSS needs another update?
The only CSS change might be a z-index change if GTK uses two mute buttons as the Safari build does (but from mediaControlsGtk.css it doesn't seem to be the case).
> I'll flag those 2 tests for now.
Otherwise, these two tests shouldn't be related to CSS changes.
Sergio Villar Senin
Comment 2
2012-05-30 10:08:13 PDT
media/video-controls-toggling.html removed from text_expectations in
http://trac.webkit.org/changeset/118930
as it's working fine now. The other test was previously removed.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug