enabling lazy frame construction for non-element editable

RESOLVED FIXED

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: m_kato, Assigned: m_kato)

Tracking

(Blocks 2 bugs)

Trunk
Unspecified
All
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 fixed)

Details

(Whiteboard: [qf:p1])

Assignee

Description

2 years ago
I am investigating enabling lazy frame construction for editable content via bug 1348073.

Except to table handling, I would like to turn on lazy frame construction for editable content such as text node.

For table, it is hard because editor and selection depend on table (cell) frame.   So, if no frame for cell, it doesn't work.

For text node, it depends on frame selection.  nsFrameSelection::MoveCaret depends on frame.  So we should adjust focus node for moving caret.
Assignee

Updated

2 years ago
Assignee: nobody → m_kato

Comment 3

2 years ago
The frame construction coming from nsGenericDOMDataNode shows up in the profiles of bug 1346723.
Blocks: 1346723

Updated

2 years ago
Whiteboard: [qf]
Whiteboard: [qf] → [qf:p1]
Assignee

Comment 4

2 years ago
I have already fixed non-text node issue such as table handling for lazy frame construction, so I will fix this with parent bug.  Although I keep open state until bug 1348073 is fixed, it will mark as fixed or wfm when bug 1348073 is fixed.

Comment 5

2 years ago
Sounds great, thank you so much Makoto!  I have wanted to fix this issue in Gecko for so many years, so glad to finally see it fixed.  :-)
Assignee

Comment 6

2 years ago
I marks as FIXED since bug 1348073 has this fix.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.