Summary: | [Qt] QtTestBrowser shows two Url input fields | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Yi Shen <max.hong.shen> | ||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ademar, commit-queue, tonikitoo | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | S60 Hardware | ||||||
OS: | S60 3rd edition | ||||||
Attachments: |
|
Description
Yi Shen
2010-10-13 12:44:44 PDT
Created attachment 70648 [details]
first try
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] first try 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. |