Bug 24946 – right clicking on selected text de-selects the text in chromium/gtk+ ports

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Tony Chang]

1) Select text on a page.
2) Right click on the selected text.
3) When the context menu appears, the text is deselected and the user is unable to copy to the clipboard because nothing is selected.

This regressed in r41715 https://bugs.webkit.org/show_bug.cgi?id=19737 .

Patch coming...

[Tony Chang]

Created attachment 29089 [details]
[1/1] Ensure the right click doesn't deselect text under it.  This was

happening in the Chromium and GTK+ ports because they don't select
the text under the cursor on right click.

This was regressed in r41715, https://bugs.webkit.org/show_bug.cgi?id=19737

https://bugs.webkit.org/show_bug.cgi?id=24946
---
 LayoutTests/ChangeLog                              |   17 ++++++++++++++
 LayoutTests/fast/events/context-no-deselect.html   |   23 ++++++++++++++++++++
 .../events/context-no-deselect-expected.checksum   |    1 +
 .../fast/events/context-no-deselect-expected.png   |  Bin 0 -> 13759 bytes
 .../fast/events/context-no-deselect-expected.txt   |   16 +++++++++++++
 WebCore/ChangeLog                                  |   17 ++++++++++++++
 WebCore/page/EventHandler.cpp                      |   10 +++++---
 7 files changed, 80 insertions(+), 4 deletions(-)

[Tony Chang]

Comment on attachment 29089 [details]
[1/1] Ensure the right click doesn't deselect text under it.  This was

Patch is missing the binary diff.  Trying to recreate the diff...

[Tony Chang]

Created attachment 29090 [details]
svn version

Same patch as before, but made with svn so the png is in the diff.

[Darin Fisher (:fishd, Google)]

Comment on attachment 29090 [details]
svn version

This change seems simple and correct to me.  r+


> Index: LayoutTests/fast/events/context-no-deselect.html
...
> +if (window.layoutTestController) {
> +     var x, y;
> +     x = input.offsetParent.offsetLeft + input.offsetLeft + input.offsetWidth / 2;
> +     y = input.offsetParent.offsetTop + input.offsetTop + input.offsetHeight / 2;
> +     eventSender.mouseMoveTo(x, y);
> +     eventSender.contextClick();

nit: indentation should be 4 spaces.  please fix this before landing.


why is it correct to copy the results from LayoutTests/platform/mac/fast/repaint/renderer-destruction-by-invalidateSelection-crash.html?  that test seems to have a very different rendering.

[Tony Chang]

(In reply to comment #4)
> why is it correct to copy the results from
> LayoutTests/platform/mac/fast/repaint/renderer-destruction-by-invalidateSelection-crash.html?
>  that test seems to have a very different rendering.

Oh, it doesn't.  I had the script create a new baseline. It was probably git that thought the file was a copy of renderer-destruction-by-invalidateSelection-crash.html.

[Tony Chang]

Created attachment 29162 [details]
v2

updated patch with indention fixed and changelog fixed.

[Tony Chang]

Can someone land this for me?  I don't have commit access.

[Darin Fisher (:fishd, Google)]

Landed as: http://trac.webkit.org/changeset/42154

I forgot to mark this as FIXED back on Wednesday :)