RESOLVED FIXED40016
[Qt] Make possible run DRT with QGraphicsWebView
https://bugs.webkit.org/show_bug.cgi?id=40016
Summary [Qt] Make possible run DRT with QGraphicsWebView
Diego Gonzalez
Reported 2010-06-01 11:56:50 PDT
Make possible to use a environment variable for switching backend. Usage: QT_DRT_WEBVIEW_MODE=graphics WebKitTools/Scripts/run-webkit-tests --qt
Attachments
Patch (6.39 KB, patch)
2010-06-01 12:16 PDT, Diego Gonzalez
no flags
Diego Gonzalez
Comment 1 2010-06-01 12:16:37 PDT
Kenneth Rohde Christiansen
Comment 2 2010-06-01 12:36:09 PDT
Comment on attachment 57578 [details] Patch Cool patch Diego :-) nice work. So what is the status about the tests? Now many are not passing? r=me, but please send an email to the list announcing this and let's wait a day before committing, just to make sure that no one have objections. Have you talked to Ossy about this already?
Csaba Osztrogonác
Comment 3 2010-06-02 09:51:30 PDT
(In reply to comment #2) > (From update of attachment 57578 [details]) > Cool patch Diego :-) nice work. So what is the status about the tests? Now many are not passing? > > r=me, but please send an email to the list announcing this and let's wait a day before committing, just to make sure that no one have objections. > > Have you talked to Ossy about this already? Without QT_DRT_WEBVIEW_MODE=graphics environment variable there aren't any regression on ToT. With this option I got similar results as Diego mentioned on webkit-qt mailing list.
Diego Gonzalez
Comment 4 2010-06-02 10:23:19 PDT
(In reply to comment #3) > Without QT_DRT_WEBVIEW_MODE=graphics environment variable > there aren't any regression on ToT. With this option I got > similar results as Diego mentioned on webkit-qt mailing list. Thanks Ossy :)
WebKit Commit Bot
Comment 5 2010-06-02 18:33:11 PDT
Comment on attachment 57578 [details] Patch Clearing flags on attachment: 57578 Committed r60586: <http://trac.webkit.org/changeset/60586>
WebKit Commit Bot
Comment 6 2010-06-02 18:33:17 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.