Bug 4707 - Need a way to automatically test for regressions in NSTextInput implementation
Summary: Need a way to automatically test for regressions in NSTextInput implementation
Status: VERIFIED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Enhancement
Assignee: Darin Adler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-28 02:52 PDT by Alexey Proskuryakov
Modified: 2005-10-10 10:04 PDT (History)
1 user (show)

See Also:

Attachments
TextInputController implementation (6.57 KB, patch)
2005-10-05 14:37 PDT, Alexey Proskuryakov 		darin: review+
Details | Formatted Diff | Diff
test the controller functionality (2.06 KB, text/html)
2005-10-05 14:38 PDT, Alexey Proskuryakov 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2005-08-28 02:52:18 PDT 
I'm not aware of any way to automatically test for regressions with issues such as as bug 4393, rdar://
4094066 or bug 4699. These seem to happen from time to time (the first two are regressions 
themselves), so having such a facility should be beneficial.
Comment 1 Alexey Proskuryakov 2005-10-05 14:37:25 PDT 
Created attachment 4222 [details]
TextInputController implementation

A first cut at implementing this facility. I haven't managed to test for
regressions in form controls (they don't want to take focus when running from
DumpRenderTree), and haven't implemented NSAttributedString support, but this
should already be quite useful for testing contentEditable text.
Comment 2 Alexey Proskuryakov 2005-10-05 14:38:15 PDT 
Created attachment 4223 [details]
test the controller functionality
Comment 3 Darin Adler 2005-10-06 07:21:58 PDT 
Comment on attachment 4222 [details]
TextInputController implementation

Looks good. Soon we might want to start breaking this tool up into multiple
files.
Comment 4 Darin Adler 2005-10-08 19:55:00 PDT 
Landed this in a separate file.