Reftest for bug 366207 wastes two seconds, needlessly

RESOLVED FIXED in mozilla6

Status

()

Core
XBL
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

Trunk
mozilla6
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

See the discussion in bug 366207 comment 10 onwards.
Created attachment 525291 [details] [diff] [review]
Patch (v1)
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #525291 - Flags: review?(bzbarsky)

Comment 2

6 years ago
Comment on attachment 525291 [details] [diff] [review]
Patch (v1)

r=me
Attachment #525291 - Flags: review?(bzbarsky) → review+
http://hg.mozilla.org/mozilla-central/rev/2e8556ead460
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
So this was tested on an older build to make sure it still triggered the assertion?
(In reply to comment #4)
> So this was tested on an older build to make sure it still triggered the
> assertion?

No.  As mentioned earlier, the test didn't really test anything useful in the first place, so there's no point in testing it on an earlier build.
Oh, why is the test there in the first place?
(In reply to comment #6)
> Oh, why is the test there in the first place?

To catch other possible future failures.  Tests are useful for other purposes behind regression testing the original change they're added for.
You need to log in before you can comment on or make changes to this bug.