Closed Bug 589933 Opened 14 years ago Closed 14 years ago

Fix racy test_bug94514.html

Categories

(Toolkit :: Places, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b5

People

(Reporter: justin.lebar+bug, Assigned: justin.lebar+bug)

References

Details

Attachments

(1 file)

A patch in bug 552023 (attachment 464835 [details] [diff] [review]) includes a fix for a race condition in test_bug94514.html.  I think this race is tickled by the patch for bug 580069.

We can fix the test without taking the whole patch for bug 552023.  I'll attach a patch in a moment.
Blocks: 552023
Attached patch Patch v1Splinter Review
sdwilsh says this has his r+ since he already r+'ed the patch that this one is copied out of.
Assignee: nobody → justin.lebar+bug
Status: NEW → ASSIGNED
Comment on attachment 468468 [details] [diff] [review]
Patch v1

r+a=sdwilsh
Attachment #468468 - Flags: review+
Attachment #468468 - Flags: approval2.0+
http://hg.mozilla.org/mozilla-central/rev/26e0dc7f707f
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Flags: in-testsuite+
Target Milestone: --- → mozilla2.0b5
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: