RESOLVED FIXED 31985
editing/selection/last-empty-inline is failing in Windows 
https://bugs.webkit.org/show_bug.cgi?id=31985
Summary editing/selection/last-empty-inline is failing in Windows
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+
DetailsFormatted DiffDiff
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.