Reset TextInputListener on nsTextEditorState::Clear()

RESOLVED FIXED in Firefox 60

Status

()

enhancement
P1
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: m_kato, Assigned: m_kato)

Tracking

Trunk
mozilla60
Points:
---

Firefox Tracking Flags

(firefox60 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

a year ago
nsTextEditorState is reusable object by HTMLInputElement.  When it is unused, it will call Unlink method to remove dependency of HTMLInputElement etc.

So we should reset TextInputListener on EditorBase too on Unlink because anyone might call nsTextEditorState from TextInputListener.  Actually, OnSelectionChange doesn't use it...
Comment hidden (mozreview-request)
Comment on attachment 8950511 [details]
Bug 1437795 - Reset TextInputListener on nsTextEditorState::Clear().

https://reviewboard.mozilla.org/r/219792/#review225506
Attachment #8950511 - Flags: review?(masayuki) → review+

Comment 3

a year ago
Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/9f075ca0c869
Reset TextInputListener on nsTextEditorState::Clear(). r=masayuki

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9f075ca0c869
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.