Bug 31985

Summary:

editing/selection/last-empty-inline is failing in Windows

Product:

WebKit

Reporter:

Shinichiro Hamaji <hamaji>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

Windows XP

URL:

https://bug-29966-attachments.webkit.org/attachment.cgi?id=43915

Attachments:

Description	Flags
Patch v1	darin: review+

Shinichiro Hamaji

Reported 2009-11-30 09:02:47 PST

I introduced (sorry!) a failing layout test (editing/selection/last-empty-inline.html) for Windows and Qt in Bug 29966. The change was: http://trac.webkit.org/changeset/51429 I put the failing test as the URL of this bug. This test drags the mouse from a point below the comma between "Hello" and "world!" and check if "world!" is selected. Before Bug 29966, "world!" wasn't selected even if we drag the mouse from the point below '!' after "world" to the comma between "Hello" and "world". The last-empty-inline was intended to check this behavior. I'll submit a patch which changes the test case so that it drags the mouse from the point below '!'. Note that it seems Qt lacks something to test editing features. So, I couldn't fix this test for now. By the way, I'm not sure if the original test case should pass as is. In Mac world, this should be definitely pass (clicking below the last line is considered as the click to the end of the last line). In Windows, I'm not sure what is the right behavior. With Firefox, "world!" is selected but nothing is selected with WebKit. WebKit ignores empty inline elements when it searches the clicked inline element even if the empty inline element is visible because of paddings. So, WebKit considers "Hello, world!" is the last line and the clicking below the last line is consider as the click to the middle of the last line in Windows. If someone think this is a bug, I'll file another bug. Hmm... my English isn't good to describe this kind of a bit complicated geometric stuff. Please feel free to say "hey, your English is terrible, elaborate again" if there are something unclear.

Attachments
Patch v1 (4.23 KB, patch) 2009-11-30 09:03 PST, Shinichiro Hamaji	darin: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Shinichiro Hamaji

Comment 1 2009-11-30 09:03:52 PST

Created attachment 44023 [details] Patch v1

Shinichiro Hamaji

Comment 2 2009-11-30 09:06:47 PST

> Patch v1 Note that I confirmed this test fails before Bug 29966 on Mac and Windows. So, I think this test is still working as the regression test of Bug 29966.

Adam Barth

Comment 3 2009-11-30 12:51:41 PST

style-queue ran check-webkit-style on attachment 44023 [details] without any errors.

Darin Adler

Comment 4 2009-12-07 10:16:04 PST

Comment on attachment 44023 [details] Patch v1 I'm not sure this is OK, but I guess I'm sure enough to rubber-stamp the change. rs=me

Shinichiro Hamaji

Comment 5 2009-12-07 21:48:55 PST

Committed r51834: <http://trac.webkit.org/changeset/51834>

Note You need to log in before you can comment on or make changes to this bug.