WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
34258
[Qt] Implement the display() method of the layout test controller
https://bugs.webkit.org/show_bug.cgi?id=34258
Summary
[Qt] Implement the display() method of the layout test controller
Benjamin Poulain
Reported
2010-01-28 06:12:55 PST
Currently, LayoutTestController::display() is not implemented in the Qt port. This make it impossible to tests problems related to painting.
Attachments
Implement the method
(4.15 KB, patch)
2010-01-28 06:16 PST
,
Benjamin Poulain
no flags
Details
Formatted Diff
Diff
Implement the method
(4.21 KB, patch)
2010-01-29 02:13 PST
,
Benjamin Poulain
no flags
Details
Formatted Diff
Diff
Move the window offscreen
(1.97 KB, patch)
2010-01-30 10:31 PST
,
Benjamin Poulain
kenneth
: review+
kenneth
: commit-queue+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Benjamin Poulain
Comment 1
2010-01-28 06:16:35 PST
Created
attachment 47611
[details]
Implement the method With this patch, the web view is shown. Calling display multiple time is not a problem, call QWidget::show() again is just ignored.
WebKit Commit Bot
Comment 2
2010-01-28 06:41:11 PST
Comment on
attachment 47611
[details]
Implement the method Clearing flags on attachment: 47611 Committed
r54000
: <
http://trac.webkit.org/changeset/54000
>
WebKit Commit Bot
Comment 3
2010-01-28 06:41:16 PST
All reviewed patches have been landed. Closing bug.
Csaba Osztrogonác
Comment 4
2010-01-28 08:45:16 PST
(In reply to
comment #3
)
> All reviewed patches have been landed. Closing bug.
Rolled out by
http://trac.webkit.org/changeset/54002
, because it broke buildbot:
http://build.webkit.org/results/Qt%20Linux%20Release/r54000%20%286648%29/results.html
Benjamin Poulain
Comment 5
2010-01-29 02:13:07 PST
Created
attachment 47691
[details]
Implement the method Some tests are unstable regarding events. It is not safe to process all events in LayoutTestController::reset(). Since we still need a synchronous paint, I have added painting on a pixmap when display() is called.
Kenneth Rohde Christiansen
Comment 6
2010-01-29 03:49:09 PST
Comment on
attachment 47691
[details]
Implement the method Sounsd like an OK idea
WebKit Commit Bot
Comment 7
2010-01-29 04:17:16 PST
Comment on
attachment 47691
[details]
Implement the method Clearing flags on attachment: 47691 Committed
r54054
: <
http://trac.webkit.org/changeset/54054
>
WebKit Commit Bot
Comment 8
2010-01-29 04:17:22 PST
All reviewed patches have been landed. Closing bug.
Andras Becsi
Comment 9
2010-01-29 16:41:07 PST
(In reply to
comment #6
)
> (From update of
attachment 47691
[details]
) > Sounsd like an OK ide
This patch has introduced an annoying effect of flickering pop-up frames and shown webviews when running the test suit. This is a regression as I see it, because the 12000 Mac tests do not show any of this behaviour. Is there a way to fix this in Qt? Kenneth, what do you think?
Kenneth Rohde Christiansen
Comment 10
2010-01-30 07:55:49 PST
Benjamin implemented this for forcing paints, and we actually dont need to show the window for doing that, now we paint to a pixmap, so that could be changed. If the behaviour is different than the other platforms we need to investigate why.
Benjamin Poulain
Comment 11
2010-01-30 08:23:27 PST
(In reply to
comment #10
)
> Benjamin implemented this for forcing paints, and we actually dont need to show > the window for doing that, now we paint to a pixmap, so that could be changed. > > If the behaviour is different than the other platforms we need to investigate > why.
Showing the window is the only way I found to catch some bugs. (In reply to
comment #9
)
> This is a regression as I see it, because the 12000 Mac tests do not show any > of this behaviour. Is there a way to fix this in Qt? Kenneth, what do you > think?
On Mac the window is shown off screen. I don't know if that works on all platform(?).
Benjamin Poulain
Comment 12
2010-01-30 10:31:58 PST
Created
attachment 47770
[details]
Move the window offscreen Here is a patch to move the window offscreen. It should work on X11, I don't know on the other systems (some window manager might reset the position on screen). Changes: + don't show the window on screen + still works with plugins and drag-and-drop - never get any expose event Note that on X11, without this patch, you can simply run a second X11 server to run the tests.
Kenneth Rohde Christiansen
Comment 13
2010-01-30 15:19:58 PST
Comment on
attachment 47770
[details]
Move the window offscreen Fine with me
Andras Becsi
Comment 14
2010-01-30 15:38:40 PST
(In reply to
comment #12
)
> Created an attachment (id=47770) [details] > Move the window offscreen
Thank you for the fast fix.
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