Closed Bug 809584 Opened 7 years ago Closed 7 years ago

Move the placeholder visibility logic to nsTextEditorState

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla19

People

(Reporter: mounir, Assigned: mounir)

References

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
It would make stuff like bug 807613 and more generally, increase readability with nearly no perf hit. The only perf hit is when we re-frame an <input> that had an editor. In that case, we could use the value passed in ::UpdateDisplayValue() but we don't.
Attachment #679317 - Flags: review?(ehsan)
Blocks: 809585
Attachment #679317 - Flags: review?(ehsan) → review+
Target Milestone: --- → mozilla19
https://hg.mozilla.org/mozilla-central/rev/9d46cc58dc1a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.