Bug 73888

Summary:

REGRESSION (WebKit2): Kill ring is not cleared when selection changes

Product:

WebKit

Reporter:

Alexey Proskuryakov <ap>

Component:

HTML Editing

Assignee:

Alexey Proskuryakov <ap>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

dglazkov, eoconnor, mrowe, webkit.review.bot

Priority:

Keywords:

InRadar, Regression

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
proposed fix	mrowe: review+, webkit.review.bot: commit-queue-

Description Alexey Proskuryakov 2011-12-05 21:24:18 PST

Pressing Ctrl+K twice should only add to kill ring if nothing else happens in between.

<rdar://problem/10532310>

Comment 1 Alexey Proskuryakov 2011-12-05 21:39:35 PST

Created attachment 117988 [details]
proposed fix

Comment 2 Alexey Proskuryakov 2011-12-05 21:41:25 PST

Comment on attachment 117988 [details]
proposed fix

Landing via commit queue to give bots time to complain.

Comment 3 WebKit Review Bot 2011-12-05 21:58:00 PST

Comment on attachment 117988 [details]
proposed fix

Attachment 117988 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10736565

Comment 4 Alexey Proskuryakov 2011-12-05 23:03:11 PST

> Attachment 117988 [details] did not pass chromium-ews (chromium-xvfb):
> Output: http://queues.webkit.org/results/10736565

The error doesn't appear related to this patch.

Comment 5 WebKit Review Bot 2011-12-06 01:59:13 PST

Comment on attachment 117988 [details]
proposed fix

Rejecting attachment 117988 [details] from commit-queue.

New failing tests:
editing/pasteboard/emacs-ctrl-k-with-move.html
Full output: http://queues.webkit.org/results/10733906

Comment 6 Alexey Proskuryakov 2011-12-06 09:38:07 PST

Committed <http://trac.webkit.org/changeset/102148>. Added the new test to fail lists of platforms that failed existing emacs tests.