[Qt] Input mode states are not reset after entering a password field

• Component: WebKit Qt
• Status: RESOLVED FIXED

[David Leong]

In Qt's implementation of void EditorClientQt::setInputMethodState(bool active), the specific input type is updated based on the input element type attribute to specify vkb handling for number, password or other special input modes.

There is a bug in this function where upon entering a password text field the input hit would be set correctly to 'hidden', but entering subsequent normal input elements the state would be stuck in the 'hidden' mode. There are other manifestations of the bug depending on the type of the text field previously entered. 

The fix is to reset the input method hints whenever EditorClientQt::setInputMethodState().

I will post up a patch as soon as possible.

[David Leong]

Made a small mistake with the description., the use case is clicking a <input type='password'> field, which sets the input mode to 'hidden'. Next click on a <textarea> and the text area input will be stuck in the password mode.

[David Leong]

Created attachment 63528 [details]
proposed change to reset input modes

The bug happens when the user clicks on a input field with a specific type then clicking on a text area. Because the input hints are never reset the text area inherits the wrong input hints.

The fix is to reset the state and set all flags from scratch.

[Laszlo Gombos]

Looks good to me.

[Simon Hausmann]

Comment on attachment 63528 [details]
proposed change to reset input modes

r=me. I'll land it manually together with a unit test.

[Simon Hausmann]

Committed r64725: <http://trac.webkit.org/changeset/64725>

[Simon Hausmann]

cherry-picked into qtwebkit-4.6 with commit bbff6182e525fcf58dbc7426a1fbbf8f31588214

[Simon Hausmann]

Revision r64725 cherry-picked into qtwebkit-2.1 with commit b0aea678e59f43c4fcc556d736da887452be5aad

[Simon Hausmann]

Revision r64725 cherry-picked into qtwebkit-2.0 with commit 31fc031c76110f1e7d8626d930559a8b5f6c8e1d