All users were logged out of Bugzilla on October 13th, 2018

Fix racy test_bug94514.html

RESOLVED FIXED in mozilla2.0b5

Status

()

RESOLVED FIXED
8 years ago
8 years ago

People

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

Tracking

Trunk
mozilla2.0b5
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
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.
(Assignee)

Updated

8 years ago
Blocks: 552023
(Assignee)

Comment 1

8 years ago
Created attachment 468468 [details] [diff] [review]
Patch v1

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+
(Assignee)

Comment 3

8 years ago
http://hg.mozilla.org/mozilla-central/rev/26e0dc7f707f
Status: ASSIGNED → RESOLVED
Last Resolved: 8 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.