Open Bug 784872 Opened 12 years ago Updated 2 years ago

test_over4GBMailboxed.js can be rewritten with nsIFileStub

Categories

(MailNews Core :: Testing Infrastructure, defect)

defect

Tracking

(Not tracked)

People

(Reporter: hiro, Unassigned)

References

(Depends on 1 open bug)

Details

(Whiteboard: [patchlove])

Attachments

(1 file)

Attached patch Proposed fix (deleted) — Splinter Review
If the nsIFileStub in bug 784870 is used, creating sparse file is unnecessary, so the test gets faster. Before: 15.47s user 2.86s system 115% cpu 15.855 total After: 1.24s user 0.13s system 91% cpu 1.503 total
Is this still needed?
Flags: needinfo?(acelists)
Whiteboard: [patchlove]
If the perf increase is real, this would be nice to have. However, the prerequisite bug is not done yet. Also, the patch here is really outdated, the 4GB test was reworked and will be rewritten again when we enable 4GB+ mbox.
Flags: needinfo?(acelists)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: