Closed
Bug 1373476
Opened 7 years ago
Closed 7 years ago
enabling lazy frame construction for non-element editable
Categories
(Core :: Layout, enhancement)
Tracking
()
People
(Reporter: m_kato, Assigned: m_kato)
References
(Blocks 2 open bugs)
Details
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•7 years ago
|
Assignee: nobody → m_kato
|
|
Assignee | |
Comment 1•7 years ago
|
||
|
|
Assignee | |
Comment 2•7 years ago
|
||
|
||
Comment 3•7 years ago
|
||
The frame construction coming from nsGenericDOMDataNode shows up in the profiles of bug 1346723.
Blocks: 1346723
|
|
||
Updated•7 years ago
|
Whiteboard: [qf]
|
||
Updated•7 years ago
|
Whiteboard: [qf] → [qf:p1]
|
|
||
Updated•7 years ago
|
Blocks: Speedometer_V2
|
|
Assignee | |
Comment 4•7 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•7 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•7 years ago
|
||
I marks as FIXED since bug 1348073 has this fix.
|
||
Updated•3 years ago
|
Performance Impact: --- → P1
Whiteboard: [qf:p1]
You need to log in
before you can comment on or make changes to this bug.
Description
•