Closed
Bug 1485934
Opened 6 years ago
Closed 6 years ago
Get rid of nsIHTMLEditor.getFontColorState since nobody uses it
Categories
(Core :: DOM: Editor, enhancement)
Core
DOM: Editor
Tracking
()
RESOLVED
FIXED
mozilla63
| Tracking | Status | |
|---|---|---|
| firefox63 | --- | fixed |
People
(Reporter: masayuki, Assigned: masayuki)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
https://searchfox.org/comm-central/search?q=getFontColorState&case=false®exp=false&path= https://github.com/therealglazou/bluegriffon/search?q=getFontColorState&unscoped_q=getFontColorState
|
Assignee | |
Comment 1•6 years ago
|
||
Neither comm-central nor BlueGriffon uses nsIHTMLEditor.getFontColorSetate(). So, we can get rid of this from nsIHTMLEditor. However, we need to keep it as a non-virtual public method since it's used by FontColorStateCommand.
|
||
Comment 2•6 years ago
|
||
Comment on attachment 9004128 [details] Bug 1485934 - Get rid of nsIHTMLEditor.getFontColorState() Makoto Kato [:m_kato] has approved the revision.
Attachment #9004128 -
Flags: review+
Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/5e3078b8f344 Get rid of nsIHTMLEditor.getFontColorState() r=m_kato
|
||
Comment 4•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/5e3078b8f344
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
|
|
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
•