Bug 65241

Summary: editing/selection/move-by-word-visually* tests timing out
Product: WebKit Reporter: Tony Gentilcore <tonyg>
Component: HTML EditingAssignee: Xiaomei Ji <xji>
Status: RESOLVED FIXED    
Severity: Normal CC: rniwa, xji
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 61344    
Attachments:
Description Flags
Patch
none
patch to split the tests none

Tony Gentilcore
Reported 2011-07-27 02:41:58 PDT
editing/selection/move-by-word-visually-others.html and editing/selection/move-by-word-visually-single-space-sigle-line.html appear to have been near the timeout threshold for some time. But after http://trac.webkit.org/changeset/91788/, they seem to timeout consistently on Windows where they never timed out before. See: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=editing%2Fselection%2Fmove-by-word-visually-others.html%2Cediting%2Fselection%2Fmove-by-word-visually-single-space-sigle-line.html
Attachments
Patch (1.37 KB, patch)
2011-07-27 02:46 PDT, Tony Gentilcore 		no flags
DetailsFormatted DiffDiff
patch to split the tests (199.64 KB, patch)
2011-07-27 11:14 PDT, Xiaomei Ji 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tony Gentilcore
Comment 1 2011-07-27 02:46:43 PDT
Created attachment 102112 [details] Patch
Tony Gentilcore
Comment 2 2011-07-27 02:50:05 PDT
Committed r91828: <http://trac.webkit.org/changeset/91828>
Tony Gentilcore
Comment 3 2011-07-27 02:52:49 PDT
Oops, leaving this bug opened to track the actual timeout. My patch just suppressed it.
Ryosuke Niwa
Comment 4 2011-07-27 09:37:59 PDT
I guess the test is too slow. We might need to split it into several pieces.
Xiaomei Ji
Comment 5 2011-07-27 11:14:51 PDT
Created attachment 102158 [details] patch to split the tests Need to improve performance eventually.
Xiaomei Ji
Comment 6 2011-07-27 12:26:27 PDT
Committed r91864: <http://trac.webkit.org/changeset/91864>
Note You need to log in before you can comment on or make changes to this bug.