Closed Bug 873322 Opened 9 years ago Closed 9 years ago

need handy function for easy getText... testing

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla24

People

(Reporter: surkov, Assigned: surkov)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
No description provided.
Attachment #750830 - Flags: review?(trev.saunders)
Comment on attachment 750830 [details] [diff] [review]
patch

This function is really to clever, but I'm not sure I have a better solution.

is there a reason you chose not to text textarea?and do you have plans to add tests for stuff involving punctuation?
Attachment #750830 - Flags: review?(trev.saunders) → review+
(In reply to Trevor Saunders (:tbsaunde) from comment #1)
> Comment on attachment 750830 [details] [diff] [review]
> patch
> 
> This function is really to clever, but I'm not sure I have a better solution.

when we get rid those todos, then we can remove the code for "function overloading"

> is there a reason you chose not to text textarea?

to avoid a lot of todos, textarea is problematic. I will investigate and file bugs.

> and do you have plans to
> add tests for stuff involving punctuation?

I think so. Adding more tests covers more problems.
https://hg.mozilla.org/mozilla-central/rev/693f3c772c0e
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.