Summary: | [Qt] [WK2] editing/inserting/typing-tab-designmode tests are failing | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Caio Marcelo de Oliveira Filho <cmarcelo> | ||||||
Component: | WebKit Qt | Assignee: | Luciano Wolf <luciano.wolf> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | luciano.wolf, webkit.review.bot | ||||||
Priority: | P2 | Keywords: | Qt, QtTriaged | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Caio Marcelo de Oliveira Filho
2012-07-23 06:32:47 PDT
Created attachment 163208 [details]
Fix TAB problem setting QKeyEvent keyText value
Comment on attachment 163208 [details] Fix TAB problem setting QKeyEvent keyText value View in context: https://bugs.webkit.org/attachment.cgi?id=163208&action=review Nice fixing that :) > Tools/ChangeLog:10 > + QKeyEvent must have keyText information. Doing an experiment with QWidget > + and intercepting it's keyPress event it's possible to notice that TAB key > + returns a keyCode (Qt::Key_Tab) + a keyText ("\t"). You can omit your experiment, and add that "Regular key events generated by Qt for the tab key fill both keyCode and keyText, so we should do the same for our fake events." > Tools/WebKitTestRunner/qt/EventSenderProxyQt.cpp:134 > + keyText = QString("\t"); Should we use QStringLiteral here? :-) http://woboq.com/blog/qstringliteral.html Created attachment 163217 [details]
Updated patch - after Caio's review
Comment on attachment 163217 [details] Updated patch - after Caio's review Clearing flags on attachment: 163217 Committed r128189: <http://trac.webkit.org/changeset/128189> All reviewed patches have been landed. Closing bug. |