Bug 38656

Summary:

[GTK] [Qt] editing/selection/shift-click.html fails

Product:

WebKit

Reporter:

Ojan Vafai <ojan>

Component:

HTML Editing

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED WORKSFORME

Severity:

Normal

CC:

abarth, chinmaya, diegohcg, eric, gustavo, mrobinson, ossy, tonikitoo

Priority:

Keywords:

Qt, QtTriaged

Version:

528+ (Nightly build)

Hardware:

OS:

Linux

Bug Depends on:

45233, 45235

Bug Blocks:

Attachments:

Description	Flags
Patch	none

Description Ojan Vafai 2010-05-06 10:27:37 PDT

As best I can tell from looking at the test results, it looks like shift+click is getting treated as a regular click. Skipping the test for now.

Comment 1 Ojan Vafai 2010-05-06 10:32:29 PDT

Created attachment 55257 [details]
Patch

Comment 2 Adam Barth 2010-05-06 10:39:24 PDT

Comment on attachment 55257 [details]
Patch

Ok, but please follow up with some of the Qt/Gtk folks by email to make sure this is how they'd like this handled.

Comment 3 Ojan Vafai 2010-05-06 10:42:43 PDT

Committed r58896: <http://trac.webkit.org/changeset/58896>

Comment 4 Martin Robinson 2010-05-06 11:36:17 PDT

My intuition says this is related to the fact that GTK+ uses Mac editing behavior by default, but this test is checking the user-agent for Mac-specific strings.

Comment 5 Ojan Vafai 2010-05-06 13:41:42 PDT

(In reply to comment #4)
> My intuition says this is related to the fact that GTK+ uses Mac editing
> behavior by default, but this test is checking the user-agent for Mac-specific
> strings.

As best I can tell, that's not the case here. The test results don't match windows or mac editing behavior. They match the behavior of doing a regular click.

Comment 6 Martin Robinson 2010-05-06 13:57:49 PDT

There are some other bugs with GTK+ EventSender around double-clicks. I have one patch for it and will try to get it ready for review tomorrow. I wouldn't be surprised if it's failing both because Qt and GTK+ use the Mac editing style and because of bugs in DRT. I'll try to follow up when I have a moment.

Comment 7 Gustavo Noronha (kov) 2010-05-07 06:15:26 PDT

Coming from the email contact. I believe Martin is the GTK+ authority of choice for such issues, so I'll let him handle the case since he's already on it =D

Comment 8 Antonio Gomes 2010-06-09 21:22:08 PDT

test is still failing on Gtk and Qt. Reopening.

Comment 9 Eric Seidel (no email) 2010-09-02 13:54:30 PDT

Status?

Comment 10 Martin Robinson 2010-09-04 15:23:17 PDT

This patch is failing on GTK+ for two reasons:
1. There is no support in the EventSender for mouseUp and mouseDown modifiers.
2. The test uses the user-agent to figure out the editing style.

I'll post patches soon.

Comment 11 Martin Robinson 2010-09-04 15:52:24 PDT

From a quick look, it appears that number 1 is also an issue with the Qt DRT.

Comment 12 Diego Gonzalez 2011-02-23 14:09:36 PST

Still reproducible at r79447

Comment 13 Martin Robinson 2015-05-07 16:47:32 PDT

This doesn't seem to be an issue any longer.