Closed
Bug 1953688
Opened 1 year ago
Closed 1 year ago
[TSF] Split `TSFTextStore.cpp` for each class definition and declaration
Categories
(Core :: DOM: UI Events & Focus Handling, task)
Tracking
()
RESOLVED
FIXED
138 Branch
| Tracking | Status | |
|---|---|---|
| firefox138 | --- | fixed |
People
(Reporter: masayuki, Assigned: masayuki)
References
Details
(Keywords: inputmethod)
Attachments
(1 external file)
It's too big file and TSFTextStore implementation is complicated. Splitting them may make it easier to read.
|
Assignee | |
Comment 1•1 year ago
|
||
It's too big and the code in it is really complicated. Therefore, I don't
like to make it bigger to implement new ITfTextStore subclass. Therefore,
this patch splits the file for TSFStaticSink, InputScopeImpl (renamed to
TSFInputScope) and the others (named TSFUtils).
This patch sorts out some logging helper methods/classes. Therefore, this
patch contains some changes in TSFTextStore.cpp.
Depends on D240970
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/c6b66a72cf59
Split `TSFTextStore.cpp` for each class definition and declaration r=m_kato
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox138:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•