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)
Testing
Reftest
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla5
People
(Reporter: mounir, Assigned: mounir)
Details
Attachments
(1 file)
852 bytes,
patch
|
dbaron
:
review+
|
Details | Diff | Splinter 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+
Assignee | ||
Comment 2•13 years ago
|
||
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.
Description
•