Closed Bug 789881 Opened 12 years ago Closed 12 years ago

[TSF] TSF isn't available when you open bookmark panel at first time

Categories

(Core :: Widget: Win32, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME
mozilla19

People

(Reporter: masayuki, Assigned: masayuki)

References

Details

(Keywords: inputmethod)

Now, we always return 0 for dwDynamicFlags for GetStatus().

However, it cannot notify our state change.

For example, at doing following steps:

1. Launch Firefox in TSF mode and the star of active tab must be stared
2. Focus to the search bar
3. Open IME on the editor
4. Type something and commit it
5. Press star button
6. Use IME on the bookmark's panel

Then, IME works in IMM mode.

The reason is, we make nsTextStore with IME disabled state. And actually focus moved to the editor on the panel, IME state becomes enabled.

In this time, we're enabling keyboard from CommitCompositionInternal(). However, it's not enough.

I think that if IME is disabled, our status should be "read-only". And when IME state becomes enabled, we should become "editable" and notify the change to TSF.
Hmm, my investigation was wrong. The status issue isn't matter for the bug.

Anyway, we need to fix the issue.
Summary: [TSF] Use proper status for GetStatus() → [TSF] TSF isn't available when you open bookmark panel at first time
The patch for bug 805766 *hides* this bug. I think nsTextStore has a bug actually.

-> WFM
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → WORKSFORME
Target Milestone: --- → mozilla19
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
relanded the patch.
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.