Bug 36368 - [Qt] Maemo5 theme - customized popup for <select multiple> elements
Summary: [Qt] Maemo5 theme - customized popup for <select multiple> elements
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Luiz Agostini
Keywords: Qt
Depends on: 36186 36769 36789
Blocks: 36372
  Show dependency treegraph
Reported: 2010-03-19 08:09 PDT by Luiz Agostini
Modified: 2010-04-06 03:09 PDT (History)
3 users (show)

See Also:

patch 1 (3.90 KB, patch)
2010-03-29 16:44 PDT, Luiz Agostini
no flags Details | Formatted Diff | Diff
patch 2 (3.87 KB, patch)
2010-03-31 11:52 PDT, Luiz Agostini
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Luiz Agostini 2010-03-19 08:09:52 PDT
Maemo5 should use multiselect popups. For it to work it is needed to check if QComboBox can be used for <select multiple> and if not a new popup class must be created. QtFallbackWebPopup must be adjusted anyway.
Comment 1 Tor Arne Vestbø 2010-03-22 06:37:00 PDT
Bugs relating to the Qt port of WebKit should have the Qt keyword. See http://trac.webkit.org/wiki/QtWebKitBugs
Comment 2 Luiz Agostini 2010-03-29 16:44:06 PDT
Created attachment 51982 [details]
patch 1
Comment 3 Luiz Agostini 2010-03-31 11:52:04 PDT
Created attachment 52192 [details]
patch 2
Comment 4 Kenneth Rohde Christiansen 2010-04-01 20:20:15 PDT
Comment on attachment 52192 [details]
patch 2

r=me, but in the future please write everything you are doing in the ChangeLog, like that you removed Maemo related code in the fallback and why that is right. I am still reviewing positively as I discussed the patch with you in person.
Comment 5 WebKit Commit Bot 2010-04-02 00:39:39 PDT
Comment on attachment 52192 [details]
patch 2

Clearing flags on attachment: 52192

Committed r56980: <http://trac.webkit.org/changeset/56980>
Comment 6 WebKit Commit Bot 2010-04-02 00:39:45 PDT
Comment on attachment 52192 [details]
patch 2

Rejecting patch 52192 from commit-queue.

Unexpected failure when landing patch!  Please file a bug against webkit-patch.
Failed to run "['WebKitTools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', 'land-attachment', '--force-clean', '--non-interactive', '--build-style=both', '--quiet', '52192', '--parent-command=commit-queue', '--no-update']" exit_code: 1
Last 500 characters of output:
Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_html.py", line 546, in __getattr__
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_html.py", line 559, in forms
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_html.py", line 228, in forms
    raise ParseError(exc)
Comment 7 Eric Seidel (no email) 2010-04-02 01:21:36 PDT
This was bug 33659.  Sorry about that.
Comment 8 Luiz Agostini 2010-04-05 07:15:07 PDT
patch is landed.
Comment 9 Luiz Agostini 2010-04-05 07:32:24 PDT
landed in r56980
Comment 10 Simon Hausmann 2010-04-06 03:09:27 PDT
Revision r56980 cherry-picked into qtwebkit-2.0 with commit 9696a3d63eef6d6aca9e5eb7724e235b57c5a9f7