Closed Bug 617273 Opened 14 years ago Closed 14 years ago

make editable text tests async

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b9

People

(Reporter: surkov, Assigned: surkov)

References

(Blocks 1 open bug)

Details

(Keywords: access)

Attachments

(1 file)

Attached patch patchSplinter Review
Bug 498015 makes editable text methods to work async in general. We need to fix mochitest to fit to this.
Attachment #495784 - Flags: review?(marco.zehe)
Attachment #495784 - Flags: review?(fherrera)
Comment on attachment 495784 [details] [diff] [review]
patch

>+    var testID = "setTextConents '" + aStr + "' for " + prettyName(aID);

"setTextContents (typo, missing t).

>+   * Proceedes the tests.

For this,

+   * Runs the tests.

is the appropriate term.

Nice to see the text test runner do all the work in a centralized place! r=me with the above fixed.
Attachment #495784 - Flags: review?(marco.zehe) → review+
Comment on attachment 495784 [details] [diff] [review]
patch

r=me with Marco comments addressed
Attachment #495784 - Flags: review?(fherrera) → review+
landed on 2.0 with Marco's comments addressed - http://hg.mozilla.org/mozilla-central/rev/19427aebdace
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b9
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: