Closed
Bug 725233
Opened 13 years ago
Closed 13 years ago
[TSF] Remove text event hack for the nsEditor's hack which will be removed by bug 622247
Categories
(Core :: Widget: Win32, defect)
Tracking
()
RESOLVED
FIXED
mozilla13
People
(Reporter: masayuki, Assigned: masayuki)
References
Details
(Keywords: inputmethod)
Attachments
(1 file)
3.05 KB,
patch
|
jimm
:
review+
|
Details | Diff | Splinter Review |
When nsTextStore needs to dispatch empty text event immediately after compositionstart, it dispatches two text events. One has a whitespace and the other has empty string. If there is no composition string, nsEditor does nothing if empty text event comes. But of course, it should replace selected text with empty string. Therefore, nsTextEditor dispatches the hacky first event. The nsEditor's behavior will be fixed by bug 622247. After that, we should remove the hacky event dispatchers.
Assignee | ||
Comment 1•13 years ago
|
||
I'm not sure how to test these cases actually, however, our internal code is tested by automated tests which will be landed by bug 622247. So, this patch must be safe.
Attachment #597269 -
Flags: review?(jmathies)
![]() |
||
Updated•13 years ago
|
Attachment #597269 -
Flags: review?(jmathies) → review+
Assignee | ||
Comment 2•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f444ebecc91f
Whiteboard: [inbound]
Target Milestone: --- → mozilla13
Comment 3•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f444ebecc91f
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•13 years ago
|
Whiteboard: [inbound]
You need to log in
before you can comment on or make changes to this bug.
Description
•