|Summary:||[Qt] QtTestBrowser: after switching between QWebView and QGraphicsWebView, rotation actions get broken|
|Product:||WebKit||Reporter:||Antonio Gomes <tonikitoo>|
|Component:||Tools / Tests||Assignee:||Antonio Gomes <tonikitoo>|
|Version:||528+ (Nightly build)|
|Bug Depends on:||43851|
Description Antonio Gomes 2010-08-11 07:44:33 PDT
When bug 43851 ([Qt] QtTestBrowser: switching between QWebView and QGraphicsWebView modes is broken) gets fixed, we also have to fix the way animatedRotation and animatedYRotation slots of WebViewGraphicsBased get connected and triggerred. Patch coming.
Comment 1 Antonio Gomes 2010-08-11 07:52:44 PDT
Created attachment 64114 [details] (committed with r65165, r=ariya) patch v1 LauncherWindow::createChrome is connecting menu itens to WebViewGraphicsBased's slots directly. It behaviors badly when user launches the QtTestBrowser in QWebView mode, since then switching to QGraphicsWebView mode, createChrome is not called again, and menu items end up not getting connected to slots at all. Patch addresses the problem.
Comment 2 Antonio Gomes 2010-08-11 09:09:25 PDT
Wrongly pressed a button. Reopening...
Comment 3 Ariya Hidayat 2010-08-11 10:42:33 PDT
Comment on attachment 64114 [details] (committed with r65165, r=ariya) patch v1 LGMT with the change to qobject_cast.
Comment 4 Antonio Gomes 2010-08-11 11:30:59 PDT
Comment on attachment 64114 [details] (committed with r65165, r=ariya) patch v1 Clearing flags on attachment: 64114 Committed r65165: <http://trac.webkit.org/changeset/65165>
Comment 5 Antonio Gomes 2010-08-11 11:31:25 PDT
(In reply to comment #3) > (From update of attachment 64114 [details]) > LGMT with the change to qobject_cast. Done. Thank you!