Closed
Bug 1274484
Opened 9 years ago
Closed 8 years ago
[TSF] Remove TestWinTSF.cpp and related code in nsWindow and TSFTextStore
Categories
(Core :: Widget: Win32, defect, P3)
Tracking
()
RESOLVED
FIXED
mozilla51
Tracking | Status | |
---|---|---|
firefox51 | --- | fixed |
People
(Reporter: masayuki, Assigned: oshinjyo14, Mentored)
Details
(Keywords: inputmethod, Whiteboard: tpi:+)
Attachments
(1 file)
93.54 KB,
patch
|
masayuki
:
review+
|
Details | Diff | Splinter Review |
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•8 years ago
|
Priority: -- → P3
Whiteboard: tpi:+
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → oshinjyo14
Mentor: masayuki
Status: NEW → ASSIGNED
Reporter | ||
Comment 2•8 years ago
|
||
Attachment #8785548 -
Flags: review?(masayuki)
Reporter | ||
Comment 3•8 years ago
|
||
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+
Reporter | ||
Comment 4•8 years ago
|
||
Reporter | ||
Comment 5•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4e837c5be1137927f8a9400beab563c6530cf23c
Bug 1274484 Removing TestWinTSF.cpp r=masayuki
Reporter | ||
Comment 6•8 years ago
|
||
# Sorry for the delay to land. I was sick yesterday.
Thank you for your contribution!
Comment 7•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Updated•8 years ago
|
status-firefox49:
affected → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•