478029 - [MSFT-7923] [MSFT-10204] [TSF] enable TSF support in default prefs on Vista or later

Status: RESOLVED FIXED

(Core :: Widget: Win32, defect)

Description

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

Jim Chen implemented TSF support on Windows in bug 88831. However, there are some blocker bugs for IME users. Therefore, currently, TSF support is disabled by default prefs.

You can test TSF support by to change "intl.enable_tsf_support" to true.

Comment 1

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

We have to fix following bugs at least for this bug.

1. bug 482138
MS Natural Input is only using TF_ATTR_OTHER for the attribute of composition string and it specifies several styles for composition string. Therefore, users can know/see the difference of meaning of each clauses of composition string by the difference of the styles. I'll finish first patch of this bug soon.

2. MS Natural Input doesn't work fine in most cases
In bugzilla-jp, some testers reported many strange bugs which are reproduced with MS Natural Input. I guess that our composition start/end timing is not enough. I think that we need to redesign the composition transaction management in nsTextStore.

3. Win2k + MS Office
I don't test this case yet. However, Japanese testers reported some critical bugs. E.g., bug 478884 and bug 478885. I need to check them. If they can fix without big changes, we can ignore them for this bug. Otherwise, they should block this bug.

Comment 2

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

The 2 is worked on bug 480708.

Comment 3

[Jim Mathies [:jimm]]

If you need any help with #1 & #2 Masayuki, I'd be more than happy to help. 2K issues are for the most part are now non-blockers for 1.9.2.

Comment 4

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

Yes, now we can ignore #3 at this bug. But we still need to check the bug after this, because that may tell us the hidden bugs on XP and later.

I think bug 482138 will be fixed in this several weeks. I don't find the goal of bug 480708 yet. But it should be found in this several weeks.

Comment 5

[Al Billings [:abillings - ex-MoCo]]

Have we considered fixing this for 1.9.1?

Comment 6

[Al Billings [:abillings - ex-MoCo]]

Adding tracking links for issues reported by Microsoft.

Comment 7

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

(In reply to comment #5)
> Have we considered fixing this for 1.9.1?

No, we gave up already. See bug 88831 comment 80. And we need to fix very many bugs until final release of our TSF support.

Comment 8

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

FYI:

TSF support bugs in bugzilla-jp.
http://bugzilla.mozilla.gr.jp/buglist.cgi?query_format=advanced&short_desc_type=allwordssubstr&short_desc=[TSF]&long_desc_type=substring&long_desc=&bug_file_loc_type=allwordssubstr&bug_file_loc=&status_whiteboard_type=allwordssubstr&status_whiteboard=&keywords_type=allwords&keywords=&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&emailassigned_to1=1&emailtype1=substring&email1=&emailassigned_to2=1&emailreporter2=1&emailqa_contact2=1&emailcc2=1&emailtype2=substring&email2=&bugidtype=include&bug_id=&chfieldfrom=&chfieldto=Now&chfieldvalue=&cmdtype=doit&order=Reuse+same+sort+as+last+time&field0-0-0=noop&value0-0-0=&type0-0-0=noop

Comment 9

[Jim Mathies [:jimm]]

Masayuki, For 3.6, could we potentially enable TSF by default for os/language combinations that don't have major unresolved issues? For example, it doesn't look like latin systems have issues, so maybe we could enable TSF for these systems that also support tablet or touch enabled displays?

Comment 10

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

No, it shouldn't be. Even if the system locale is not CJK, the systems can be used CJK local IMEs. And some users use them.

We should disable TSF on 3.6. By the 1.9.2 release date was changed to this fall, I needed to change the roadmap of my works, unfortunately.

Comment 11

[David Gaskill]

Vista has come and gone and Windows speech recognition never worked in Firefox or Thunderbird under that operating system.  I filed bug report 395484 in September 2007 - getting on for four years ago.

Windows 7 has been current for around two years and still TSF isn't working in either of the above Mozilla applications.  

The last posting in this thread was nearly two years ago.

Even if I enable TSF speech recognition doesn't work well enough to be useful.  At the suggestion of  Jim Mathies I have filed a separate bug report, 650833, on the loss of focus.

It won't be so long before Windows 8 is with us and as Jim has pointed out, "tablet / touch input support is going to be *real* important with Win8"

Is it possible to provide an estimate of when work might start on fixing the problem both Firefox and Thunderbird?  Hopefully this will be before Windows 9 is upon us…

Comment 12

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

I think that when we enable TSF in default settings, we should keep disabling TSF on prior to Vista because the TSF framework on XP has a lot of different behavior. Additionally, MS's Natural Input 2002 which is bundled and behaves much complicated.

Comment 13

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

(In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #12)
> Additionally, MS's Natural Input 2002 which is bundled
> and behaves much complicated.

I meant that it's bundled to only XP and it behaves much complicated. So, it's hard to support it and XP will be end of its life cycle even though we'll keep supporting it.

Comment 14

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

Attachment: Enable TSF in release builds

Comment 15

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

Comment on attachment 8592143 [details] [diff] [review]
Enable TSF in release builds

Right now, the final blocker of this bug has been fixed!

Let's enable TSF in release builds.

Currently, we've already enabled TSF mode in Aurora (38). Therefore, it's now being tested wider than 37. If somebody will file serious bugs of TSF mode, we can back this out from future branches or m-c.

So, I think that it is the time to land this patch!

Comment 16

[Jim Mathies [:jimm]]

Comment on attachment 8592143 [details] [diff] [review]
Enable TSF in release builds

How is TSF behaving with e10s currently?

Comment 17

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

(In reply to Jim Mathies [:jimm] from comment #16)
> Comment on attachment 8592143 [details] [diff] [review]
> Enable TSF in release builds
> 
> How is TSF behaving with e10s currently?

With e10s, TSF is disabled. When you enable TSF mode with e10s mode, you need to enable "intl.tsf.force_enable". See bug 1147722.

Comment 18

[Masatoshi Kimura [:emk]]

Comment on attachment 8592143 [details] [diff] [review]
Enable TSF in release builds

\o/

Comment 19

[Makoto Kato [:m_kato]]

Comment on attachment 8592143 [details] [diff] [review]
Enable TSF in release builds

great!

Comment 20

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c0c4d56594f9

Comment 21

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/c0c4d56594f9

Comment 22

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

Release Note Request (optional, but appreciated)
[Why is this notable]: TSF support is a big change for IME users of Windows because it supports a lot of features which cannot be supported with IMM. And also some a11y tools can use TSF for querying editing contents.
[Suggested wording]: Firefox 41 supports TSF (Text Services Framework). This provides better UX for IME users of Windows.
[Links (documentation, blog post, etc)]: Nothing.

# TSF is new IME framework on Windows. This allows IME to access full contents of editing field. So, they can improve conversion result from surrounding text. And also we can give native look of composition string with TSF's new feature.

Comment 23

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Added to FF41 release notes.

Comment 24

[Florian Bender]

As many are probably unaware what TSF is, please change the relnotes thusly:

[Suggested wording]: Enhance IME support on Windows (Vista +) using TSF (Text Services Framework)

Comment 25

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

(In reply to Florian Bender from comment #24)
> As many are probably unaware what TSF is, please change the relnotes thusly:
> 
> [Suggested wording]: Enhance IME support on Windows (Vista +) using TSF
> (Text Services Framework)

Done.