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 ***