Closed
Bug 1363278
Opened 8 years ago
Closed 8 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•8 years ago
|
||
Comment 2•8 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•8 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•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee | ||
Updated•5 years ago
|
Blocks: redesign-editor-scriptable-API
You need to log in
before you can comment on or make changes to this bug.
Description
•