|Summary:||[Qt] Multiselect Popup - adjusting QtAbstractWebPopup|
|Product:||WebKit||Reporter:||Luiz Agostini <luiz>|
|Component:||WebCore Misc.||Assignee:||Nobody <webkit-unassigned>|
|Severity:||Normal||CC:||commit-queue, hausmann, kenneth, koivisto|
|Version:||528+ (Nightly build)|
|Bug Depends on:||36178|
Description Luiz Agostini 2010-03-16 13:41:51 PDT
QtAbstractWebPopup must provide new methods to be used by its descendants that wish to handle <select multiple> elements.
Comment 3 Kenneth Rohde Christiansen 2010-03-19 21:35:34 PDT
Comment on attachment 51174 [details] patch 2 I assume this doesn't break normal non-multiple selects as valueChanged() is not never being called. Please confirm that before adding this to the cq.
Comment 4 Tor Arne Vestbø 2010-03-22 06:36:58 PDT
Bugs relating to the Qt port of WebKit should have the Qt keyword. See http://trac.webkit.org/wiki/QtWebKitBugs
Comment 5 Luiz Agostini 2010-03-22 10:15:15 PDT
(In reply to comment #3) > (From update of attachment 51174 [details]) > I assume this doesn't break normal non-multiple selects as valueChanged() is > not never being called. Please confirm that before adding this to the cq. It doesn't. :)
Comment 6 WebKit Commit Bot 2010-03-22 10:38:53 PDT
Comment on attachment 51174 [details] patch 2 Clearing flags on attachment: 51174 Committed r56341: <http://trac.webkit.org/changeset/56341>
Comment 7 WebKit Commit Bot 2010-03-22 10:38:58 PDT
All reviewed patches have been landed. Closing bug.