Bug 65241 - editing/selection/move-by-word-visually* tests timing out
Summary: editing/selection/move-by-word-visually* tests timing out
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Xiaomei Ji
URL:
Keywords:
Depends on:
Blocks: 61344
  Show dependency treegraph
 
Reported: 2011-07-27 02:41 PDT by Tony Gentilcore
Modified: 2011-07-27 12:26 PDT (History)
2 users (show)

See Also:


Attachments
Patch (1.37 KB, patch)
2011-07-27 02:46 PDT, Tony Gentilcore
no flags Details | Formatted Diff | Diff
patch to split the tests (199.64 KB, patch)
2011-07-27 11:14 PDT, Xiaomei Ji
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Gentilcore 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
Comment 1 Tony Gentilcore 2011-07-27 02:46:43 PDT
Created attachment 102112 [details]
Patch
Comment 2 Tony Gentilcore 2011-07-27 02:50:05 PDT
Committed r91828: <http://trac.webkit.org/changeset/91828>
Comment 3 Tony Gentilcore 2011-07-27 02:52:49 PDT
Oops, leaving this bug opened to track the actual timeout. My patch just suppressed it.
Comment 4 Ryosuke Niwa 2011-07-27 09:37:59 PDT
I guess the test is too slow.  We might need to split it into several pieces.
Comment 5 Xiaomei Ji 2011-07-27 11:14:51 PDT
Created attachment 102158 [details]
patch to split the tests

Need to improve performance eventually.
Comment 6 Xiaomei Ji 2011-07-27 12:26:27 PDT
Committed r91864: <http://trac.webkit.org/changeset/91864>