Bug 47548 – [Qt] editing/input/emacs-ctrl-o.html

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Robert Hogan]

Qt is missing a couple of editing delegate messages from the Mac expected results:

EDITING DELEGATE: shouldChangeSelectedDOMRange:range from 0 of DIV > DIV > BODY > HTML > #document to 0 of DIV > DIV > BODY > HTML > #document toDOMRange:range from 9 of #text > DIV > DIV > BODY > HTML > #document to 9 of #text > DIV > DIV > BODY > HTML > #document affinity:NSSelectionAffinityDownstream stillSelecting:FALSE
EDITING DELEGATE: webViewDidChangeSelection:WebViewDidChangeSelectionNotification

Qt is missing these because ctrl-o is bound to a sequence of commands on Mac, namely: insertParagraphSeparator then moveUp. In the DRT, we just interepret ctrl-o in EventSenderQt as '\n', i.e. insertParagraphSeparator. As a result we only issue one command, so don't generate the change in caret position that results in the above editing delegate messages.

Since we are only passing an insertParagraphSeparator to mimic other DRTs and Qt clients are free to re-implement whatever key binding they like to perform whatever editing actions they like there is no point in bending over backwards to get DRT, or even EditorClientQt to mimic Mac here by repositioning the caret. So just add our own expected results and unskip.

[Robert Hogan]

Created attachment 70556 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 70556 [details]
Patch

Clearing flags on attachment: 70556

Committed r69597: <http://trac.webkit.org/changeset/69597>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.