We can input longer text by IME on <input> which has maxlength and length of the value has been same as maxlength already

RESOLVED FIXED in mozilla6

Status

()

Core
Editor
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

({inputmethod, regression})

Trunk
mozilla6
inputmethod, regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

Created attachment 531221 [details] [diff] [review]
Patch v1.0

1. Open the URL (data:text/html,<input maxlength="1" value="a">).
2. Type something with IME.
3. Commit the composition string.

Then, you can see uncommitted string on the <input> but actually, the composition string was committed.

When editor discards all characters at committing the composition string, it doesn't clean up the textnode for composition string. See the patch for the detail.

This is a regression between Fx3.6 and Fx4. The regression range is between 20101007 and 20101008. I'm not sure the actual bug#.
Attachment #531221 - Flags: review?(ehsan)

Updated

6 years ago
Blocks: 597519
Comment on attachment 531221 [details] [diff] [review]
Patch v1.0

Can you please change the comment to say something like:

// If we're exceeding the maxlength when composing IME, we need to clean up
// the composing text, so we shouldn't return early.

r=me with that.
Attachment #531221 - Flags: review?(ehsan) → review+
http://hg.mozilla.org/mozilla-central/rev/64f5f44a8334
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.