Closed
Bug 1363278
Opened 7 years ago
Closed 7 years ago
Get rid of nsIPhonetic and MozPhonetic.phonetic (HTMLInputElement.phonetic) because of unused
Categories
(Core :: DOM: Editor, enhancement)
Core
DOM: Editor
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: masayuki, Assigned: masayuki)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete, inputmethod)
Attachments
(1 file)
Looks like that nsIPhonetic is not used anybody (including add-ons and BlueGriffon). IIRC, it was used for a11y to retrieve phonetic of active composition string. However, the implementation was really hacky: https://searchfox.org/mozilla-central/rev/224cc663d54085994a4871ef464b7662e0721e83/editor/libeditor/EditorBase.cpp#2526,2528-2532,2537-2539 So, I guess that a11y stop using it with some reason (e.g., giving up to implement it or using new API on some platforms).
Assignee | ||
Comment 1•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=629a4b9758fc5c548b436834d22d6c60f6eb55a2
Comment 2•7 years ago
|
||
Marking this instantly as dev-doc-complete, as we never documented this on MDN anyway, therefore no docs work is needed.
Keywords: dev-doc-complete
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8866236 [details] Bug 1363278 Get rid of nsIPhonetic and HTMLInputElement.phonetic https://reviewboard.mozilla.org/r/137864/#review141036 ::: dom/html/HTMLInputElement.h (Diff revision 1) > > class HTMLInputElement final : public nsGenericHTMLFormElementWithState, > public nsImageLoadingContent, > public nsIDOMHTMLInputElement, > public nsITextControlElement, > - public nsIPhonetic, Thanks! This reduces the size of input elements by 4 or 8 bytes (32bit vs 64bit systems).
Attachment #8866236 -
Flags: review?(bugs) → review+
Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/57330642868c Get rid of nsIPhonetic and HTMLInputElement.phonetic r=smaug
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/57330642868c
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee | ||
Updated•4 years ago
|
Blocks: redesign-editor-scriptable-API
You need to log in
before you can comment on or make changes to this bug.
Description
•