Closed Bug 1275910 Opened 8 years ago Closed 8 years ago

[TSF] Perhaps, TSFTextStore shouldn't notify TSF of selection change after notifying TSF of text change

Categories

(Core :: Widget: Win32, defect)

Product:
Core
Component:
Widget: Win32
Platform:
All
Windows
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox49 --- affected

People

(Reporter: masayuki, Assigned: masayuki)

References

Details

(Keywords: inputmethod, perf) 

I realized that when we notify TSF/TIP of text change, TSF/TIP must always check new selection because keeping believing selection in old contents doesn't make sense.

So, I think that TSFTextStore should check if selection is retrieved by TSF/TIP during notifying TSF/TIP of a text change, we shouldn't notify TSF/TIP of selection change.

This must reduce a lot of WidgetQueryContentEvent cost.
OS: All → Windows
The fix of bug 1224994 makes it this behavior unexpectedly.

However, at least for Google Japanese Input, looks like it has trouble. Anyway, I'll investigate it on other bug.
Status: NEW → RESOLVED
Closed: 8 years ago
Depends on: 1224994
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.