Closed
Bug 378018
Opened 18 years ago
Closed 17 years ago
Need an automated test for <textbox type="number">
Categories
(Toolkit :: XUL Widgets, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9beta1
People
(Reporter: enndeakin, Assigned: enndeakin)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
asaf
:
review+
|
Details | Diff | Splinter Review |
Bug 368097 has a test for generic textboxes which can be used as a basis for this.
Assignee | ||
Comment 1•18 years ago
|
||
Needs window to be focused to work properly.
Used the DOMWindowUtils.send(Mouse/Key)Event functions here, but these calls should be put somewhere else.
Assignee | ||
Comment 2•18 years ago
|
||
sayrer, where should I add the event functions, or are there already some calls that use the DOMWindowUtils events simulation?
Status: NEW → ASSIGNED
Comment 3•18 years ago
|
||
(In reply to comment #2)
> sayrer, where should I add the event functions, or are there already some calls
> that use the DOMWindowUtils events simulation?
>
There's an EventUtils.js file in testing/mochitest/tests/SimpleTest. It has sendKey and things that predate your DOMWindowUtils code. I'm not sure if we should replace that stuff with your new method or just add to it. You and bz (who wrote the original EventUtils.js) can make a more informed decision than I can.
Comment 4•18 years ago
|
||
Yeah, I think the existing EventUtils key-sending code does what you want here (and also handles people cancelling part of the keypress, etc).
Assignee | ||
Updated•17 years ago
|
Target Milestone: --- → mozilla1.9beta2
Assignee | ||
Comment 5•17 years ago
|
||
Attachment #262156 -
Attachment is obsolete: true
Attachment #276786 -
Flags: review?(mano)
Comment 6•17 years ago
|
||
Comment on attachment 276786 [details] [diff] [review]
updated numberbox tests
r=mano
Attachment #276786 -
Flags: review?(mano) → review+
Assignee | ||
Updated•17 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•