[TSF] Remove TestWinTSF.cpp and related code in nsWindow and TSFTextStore

RESOLVED FIXED in Firefox 51

Status

()

Core
Widget: Win32
P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: masayuki, Assigned: ou shinjo, Mentored)

Tracking

({inputmethod})

Trunk
mozilla51
All
Windows
inputmethod
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

(Whiteboard: tpi:+)

Attachments

(1 attachment)

TestWinTSF.cpp was disabled at several years ago. And it was designed for keeping TSFTextStore (nsTextStore)'s behavior rather than emulating each behavior of existing TIP. So, we don't need it anymore.

Ideally, we should create a new scriptable interface which for emulating TSF/TIP with JS but it's not a scope of this bug.

Updated

2 years ago
Priority: -- → P3
Whiteboard: tpi:+
(Assignee)

Comment 1

2 years ago
Created attachment 8785548 [details] [diff] [review]
Removing TestWinTSF.cpp
Assignee: nobody → oshinjyo14
Mentor: masayuki
Status: NEW → ASSIGNED
(Assignee)

Updated

2 years ago
Attachment #8785548 - Flags: review?(masayuki)
Comment on attachment 8785548 [details] [diff] [review]
Removing TestWinTSF.cpp

Thank you for your work! I'll land this soon.
Attachment #8785548 - Flags: review?(masayuki) → review+
# Sorry for the delay to land. I was sick yesterday.

Thank you for your contribution!

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/4e837c5be113
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
status-firefox49: affected → ---
You need to log in before you can comment on or make changes to this bug.