Closed Bug 1373476 Opened 4 years ago Closed 4 years ago

enabling lazy frame construction for non-element editable

Categories

(Core :: Layout, enhancement)

Unspecified
All
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
firefox57 --- fixed

People

(Reporter: m_kato, Assigned: m_kato)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [qf:p1])

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: nobody → m_kato
The frame construction coming from nsGenericDOMDataNode shows up in the profiles of bug 1346723.
Blocks: 1346723
Whiteboard: [qf]
Whiteboard: [qf] → [qf:p1]
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.
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.  :-)
I marks as FIXED since bug 1348073 has this fix.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.