Closed
Bug 292418
Opened 19 years ago
Closed 19 years ago
rtest log should be brief
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bernd_mozilla, Assigned: bernd_mozilla)
Details
Attachments
(1 file)
|
1.06 KB,
patch
|
dbaron
:
review+
dbaron
:
superreview+
|
Details | Diff | Splinter Review |
currently a typical rtest run produces a 5MB textfile with 4 false positives as
we dump the complete frames on error. I think this is redundant. As the brief
log points to the differing frames and one can then compare the rgd failes for
this testcase if one really wnat to completely understand the failure.
Attachment #182240 -
Flags: superreview?(dbaron)
Attachment #182240 -
Flags: review?(dbaron)
Comment on attachment 182240 [details] [diff] [review]
patch
r+sr=dbaron, but cc:ing Boris and Robert so they have a chance to object if
they want
Attachment #182240 -
Flags: superreview?(dbaron)
Attachment #182240 -
Flags: superreview+
Attachment #182240 -
Flags: review?(dbaron)
Attachment #182240 -
Flags: review+
|
||
Comment 3•19 years ago
|
||
No, this sounds good to me. Much more readable that way.
no objection
fix checked in
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Product: Core → Core Graveyard
|
||
Updated•6 years ago
|
Component: Layout: Misc Code → Layout
Product: Core Graveyard → Core
You need to log in
before you can comment on or make changes to this bug.
Description
•