Closed Bug 875201 Opened 11 years ago Closed 11 years ago

move getText at word boundary tests into test_wordboundery.html

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla24

People

(Reporter: surkov, Assigned: surkov)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
      No description provided.
Attachment #753101 - Flags: review?(trev.saunders)
Assignee: nobody → surkov.alexander
Comment on attachment 753101 [details] [diff] [review]
patch

So, one disadvantage of having these test functions htat have both passing and failing tests is that its harder to debug the failing ones because you have to skip the passing ones, but I guess if you know what the test is checking you can just add your own simpler test.
Attachment #753101 - Flags: review?(trev.saunders) → review+
(In reply to Trevor Saunders (:tbsaunde) from comment #1)
> Comment on attachment 753101 [details] [diff] [review]
> patch
> 
> So, one disadvantage of having these test functions htat have both passing
> and failing tests is that its harder to debug the failing ones because you
> have to skip the passing ones, but I guess if you know what the test is
> checking you can just add your own simpler test.

true, I just wanted to avoid human factor when you specify boundary offsets.
https://hg.mozilla.org/mozilla-central/rev/82749b1d775b
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: