|Summary:||fast/repaint/4774354 is failing|
|Product:||WebKit||Reporter:||Adam Roben (:aroben) <aroben>|
|Component:||Tools / Tests||Assignee:||Adam Roben (:aroben) <aroben>|
|Severity:||Normal||CC:||aroben, robert, timothy|
|OS:||OS X 10.4|
Description Adam Roben (:aroben) 2006-10-18 00:58:39 PDT
r17094 seems to have broken fast/repaint/4774354
Comment 1 David Harrison 2006-10-18 08:12:08 PDT
r17094 was for <rdar://problem/4776665> REGRESSION: In Mail, caret is visible when the message body isn't focused
Comment 2 David Harrison 2006-10-18 08:14:12 PDT
The broken test is for <rdar://problem/4774354> REGRESSION: In Gmail, caret reappears when dragging a selection over some text in the message body
Comment 3 David Harrison 2006-10-18 13:16:15 PDT
The detail of the failure is the addition of an empty RenderText node. The change in r17094 was strictly for the drawing of caret, and does not affect the render tree. Confirmed that backing out the patch did not result in a successful test.
Comment 4 Adam Roben (:aroben) 2006-10-18 14:03:51 PDT
The test was working here: http://build.webkit.org/results/post-commit-powerpc-mac-os-x/3875/results.html and broken here: http://build.webkit.org/results/post-commit-powerpc-mac-os-x/3877/results.html
Comment 5 Adam Roben (:aroben) 2006-10-18 14:05:28 PDT
Build details for those builds are here: http://build.webkit.org/post-commit-powerpc-mac-os-x/builds/3875 http://build.webkit.org/post-commit-powerpc-mac-os-x/builds/3877 As you can see, the only revisions added between those two runs were r17093 and r17094. Perhaps it was r17093 that did it?
Comment 6 Adam Roben (:aroben) 2006-10-18 18:05:11 PDT
Wow, I was too tired when triaging these failures last night. This test has been failing since its introduction. Seems like it was checked in with wrong results. This can't even really be called a regression.
Comment 7 Adam Roben (:aroben) 2006-10-18 18:08:24 PDT
Created attachment 11140 [details] New results
Comment 8 Maciej Stachowiak 2006-10-18 18:09:58 PDT
Comment on attachment 11140 [details] New results r=me
Comment 10 Robert Hogan 2013-04-07 03:39:39 PDT
Reopening to attach new patch.
Comment 11 Timothy Hatcher 2013-04-07 05:47:17 PDT