Bug 119350
| Summary: | Speed up moving cursor/selection up or down past non-renderered elements | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
| Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Normal | CC: | arpitabahuguna, darin, enrica, kling, koivisto |
| Priority: | P2 | Keywords: | BlinkMergeCandidate |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ryosuke Niwa
Merge https://chromium.googlesource.com/chromium/blink/+/965746ed22a5e210eb8c33adf300deab9a9c8332
Skips calling firstPositionInOrBeforeNode when a node doesn't have a
renderer, since there couldn't possibly be a visible position there.
This was particularly wasteful when trying to move the cursor up when at
the beginning of a document, as it did a O(n^2) scan through the document head.
See bug for easy repro steps.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Arpita Bahuguna
http://trac.webkit.org/changeset/154274 fixes this issue.
Darin Adler
*** This bug has been marked as a duplicate of bug 119882 ***