Bug 32788

Summary: [GTK] Failing test platform/gtk/editing/pasteboard/middle-button-paste.html
Product: WebKit Reporter: Alejandro G. Castro <alex>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Proposed patch
none
Proposed patch
none
Proposed patch none

Description Alejandro G. Castro 2009-12-20 03:57:04 PST
The test is failing in the bots.
Comment 1 Alejandro G. Castro 2009-12-20 04:00:13 PST
Created attachment 45269 [details]
Proposed patch

Added fixed font size.
Comment 2 WebKit Review Bot 2009-12-20 04:05:08 PST
style-queue ran check-webkit-style on attachment 45269 [details] without any errors.
Comment 3 Gustavo Noronha (kov) 2009-12-20 04:09:09 PST
Comment on attachment 45269 [details]
Proposed patch

Looks good (and works for me).
Comment 4 WebKit Commit Bot 2009-12-20 04:18:41 PST
Comment on attachment 45269 [details]
Proposed patch

Clearing flags on attachment: 45269

Committed r52413: <http://trac.webkit.org/changeset/52413>
Comment 5 WebKit Commit Bot 2009-12-20 04:18:47 PST
All reviewed patches have been landed.  Closing bug.
Comment 6 Alejandro G. Castro 2009-12-20 06:56:05 PST
This fix does not fix the problem, we have to create a test that does not use fixed values for the events positions.
Comment 7 Alejandro G. Castro 2009-12-20 07:06:40 PST
Created attachment 45275 [details]
Proposed patch

Avoid using fixed mouse positions, it was causing problems in the bots, and add button to the click count function to avoid double paste with different buttons.
Comment 8 WebKit Review Bot 2009-12-20 07:10:10 PST
style-queue ran check-webkit-style on attachment 45275 [details] without any errors.
Comment 9 Alejandro G. Castro 2009-12-20 07:24:39 PST
Created attachment 45278 [details]
Proposed patch

Small nitpick.
Comment 10 WebKit Review Bot 2009-12-20 07:25:42 PST
style-queue ran check-webkit-style on attachment 45278 [details] without any errors.
Comment 11 Gustavo Noronha (kov) 2009-12-20 08:17:59 PST
Comment on attachment 45278 [details]
Proposed patch

This should work better, thanks =). Tested locally, let's see if the bots like it.
Comment 12 WebKit Commit Bot 2009-12-20 08:26:24 PST
Comment on attachment 45278 [details]
Proposed patch

Clearing flags on attachment: 45278

Committed r52419: <http://trac.webkit.org/changeset/52419>
Comment 13 WebKit Commit Bot 2009-12-20 08:26:30 PST
All reviewed patches have been landed.  Closing bug.