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

Status: RESOLVED WORKSFORME

(Core :: Widget: Win32, defect)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

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.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Hmm, my investigation was wrong. The status issue isn't matter for the bug.

Anyway, we need to fix the issue.

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

The patch for bug 805766 *hides* this bug. I think nsTextStore has a bug actually.

-> WFM

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

relanded the patch.