Closed Bug 1771060 Opened 1 year ago Closed 1 year ago
[CTW] Caching lines is slow with large blocks
(Core :: Disability Access APIs, defect)
(Reporter: Jamie, Assigned: Jamie)
(Blocks 1 open bug)
48 bytes, text/x-phabricator-request
|Details | Review|
Spun off bug 1770606.
When sending the cache for a subtree containing a large block, IsLocalAccAtLineStart is slow because it does a linear walk of the lines in the block. With the cursor changes in bug 1770981, we can optimise this by using the line cursor. This way, subsequent calls to fetch subsequent lines in the same block will be much faster.
This will require switching to nsBlockInFlowLineIterator, since nsLineIterator doesn't support the cursor. We don't actually care about line numbers anyway, so nsLineIterator isn't necessary.
1 year ago
nsLineIterator doesn't use the line cursor, so switch to nsBlockInFlowLineIterator.
We don't actually care about the line number here anyway.
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/fc89774bca04 Make use of the line cursor in IsLocalAccAtLineStart to improve performance. r=jfkthame
1 year ago
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox102: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch
You need to log in before you can comment on or make changes to this bug.