[Webkit-unassigned] [Bug 25172] New: Extra partial layout during the first keypress in an empty block

[bugzilla-daemon at webkit.org]

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

           Summary: Extra partial layout during the first keypress in an
                    empty block
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Macintosh
        OS/Version: Mac OS X 10.5
            Status: NEW
          Keywords: InRadar
          Severity: Normal
          Priority: P2
         Component: HTML Editing
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: justin.garcia at apple.com


Within InsertTextCommand::input, it looks like on every normal keypress, we do
one layout. On the first keypress, we will layout twice.

On a normal keypress, the progression is normally:
  insertTextIntoNode(textNode, offset, text) => setNeedsLayout
  setEndingSelection(Selection(endingSelection().end(),
endingSelection().affinity()))) => updateLayout

On the first keypress, this is now:
  startPosition = prepareForTextInsertion(startPosition) => setNeedsLayout
  removePlaceholderAt(VisiblePosition(startPosition)); => updateLayout
then
  insertTextIntoNode(textNode, offset, text) => setNeedsLayout
  setEndingSelection(Selection(endingSelection().end(),
endingSelection().affinity()))) => updateLayout

<rdar://problem/6777744>


-- 
Configure bugmail: https://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.