[Webkit-unassigned] [Bug 15969] New: Eliminate Editor::deleteRange()

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=15969

           Summary: Eliminate Editor::deleteRange()
           Product: WebKit
           Version: 525+ (Nightly build)
          Platform: Macintosh
        OS/Version: Mac OS X 10.4
            Status: NEW
          Severity: Normal
          Priority: P3
         Component: HTML Editing
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: ap at webkit.org


There are a lot of deleteThisOrThat functions and classes around, too many to
my liking. Editor::deleteRange() seems to be designed as a
bottleneck/dispatcher for those, but doesn't help in my opinion:
- It's called incorrectly, as it doesn't work with ranges. It works with
selections.
- It tries to unify too many different operations - as a result, it has to
re-select what is already selected etc. Makes data flow hard to follow.
- It isn't a bottleneck anyway, as many code paths bypass it.

I think we can get rid of it. Patch forthcoming.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.