[Webkit-unassigned] [Bug 261984] New: [Performance Optimization] Skip non-text SVG elements in Most{For/Back}wardCaretPosition

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 22 16:25:43 PDT 2023


https://bugs.webkit.org/show_bug.cgi?id=261984

            Bug ID: 261984
           Summary: [Performance Optimization] Skip non-text SVG elements
                    in Most{For/Back}wardCaretPosition
           Product: WebKit
           Version: Safari Technology Preview
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Keywords: Performance
          Severity: Normal
          Priority: P2
         Component: HTML Editing
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ahmad.saleem792 at gmail.com
                CC: rniwa at webkit.org, wenson_hsieh at apple.com

Hi Team,

While going through Blink bug, I came across another potential issue, which we face as well and we are slow:

Test Case (Copy paste from crbug.com/822312):

1. Access website http://cwsdemo.hexagonusfederal.com/cwe
2. Click Open
3. Click Choose File
4. Browse to 5672-1.svg (attached)
5. Click open
6. Click Insert Item
7. Click Text
8. Click in map
9. "New Text" should be placed in the map

What is the expected behavior?
Immediate placement of default text.

What went wrong?
Depending on the size of the SVG file it takes seconds to minutes to place the text in the map with the last two official builds of Chrome.  

___

Blink Commit: https://chromium.googlesource.com/chromium/src.git/+/8e793a18ec6358414ffc337b336c93d4f6061c91

GitHub WebKit Source: https://github.com/WebKit/WebKit/blob/main/Source/WebCore/dom/Position.cpp#L715 & https://github.com/WebKit/WebKit/blob/main/Source/WebCore/dom/Position.cpp#L823

__

Just wanted to raise so we can track and see if we can apply this optimization without any regression. Haven't tried fixing on my local build (Xcode 15 curse on macOS Ventura - waiting for Sonoma).

Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230922/038aa0d7/attachment.htm>


More information about the webkit-unassigned mailing list