Closed Bug 640545 Opened 13 years ago Closed 13 years ago

Specify that the load has failed in the error message when a reftest load failed

Categories

(Testing :: Reftest, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla5

People

(Reporter: mounir, Assigned: mounir)

Details

Attachments

(1 file)

Attached patch Patch v1Splinter Review
It prevents seeing things like:
REFTEST TEST-UNEXPECTED-FAIL | http://localhost:4444/1299714845703/14/root-background-1.html | null
And would show:
REFTEST TEST-UNEXPECTED-FAIL | http://localhost:4444/1299714845703/14/root-background-1.html | load failed: null

I will see if i can try to write another patch that tries to guess he reason (or show a generic one) when none is specified.
Attachment #518340 - Flags: review?(dbaron)
Comment on attachment 518340 [details] [diff] [review]
Patch v1

r=dbaron
Attachment #518340 - Flags: review?(dbaron) → review+
Pushed:
http://hg.mozilla.org/mozilla-central/rev/5a24cad1dd8a
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: