I saw two Url input fields show on the toolbar of QtTestBrowser. In MainWindow::buildUI(), it deletes the old toolbar instance which is supposed to delete the LocationEdit as well, however, it doesn't. To make a quick fix instead of waiting Qt UI code's change, we can explicitly delete the locationedit by ourself on Symbian.
Created attachment 70648 [details]
Does it happens on trunk at all?
I think so, it can be reproduced on Symbian with Qt 4.7
(In reply to comment #2)
> Does it happens on trunk at all?
because on trunk we have this:
urlEdit = new LocationEdit(m_toolBar);
so when we delete m_toolBar we should also be deleting the urlEdit. Could you please check if the 2.1 branch has this code?
Yes, 2.1 branch has that code. (Remember we have cheery-picked your patches)
I think the root cause for this issue is from Qt Core/Gui, but we may want to address it quickly by deleting LocationEdit by ourself.
I can not reproduce this problem with trunk with qtwebkit on mac :(
what steps are you using?
It is only can be reproduced on Symbian :)
(In reply to comment #7)
> It is only can be reproduced on Symbian :)
oh :) ok
Thanks, Antonio :)
Comment on attachment 70648 [details]
Clearing flags on attachment: 70648
Committed r69714: <http://trac.webkit.org/changeset/69714>
All reviewed patches have been landed. Closing bug.
Revision r69714 cherry-picked into qtwebkit-2.1 with commit 995f39c <http://gitorious.org/webkit/qtwebkit/commit/995f39c>
duh, should've been "block" 39121, not "depends on". :-)
Anyway, already integrated.