Closed
Bug 1139021
Opened 10 years ago
Closed 10 years ago
Don't do extra GCs and CCs in Mochitest browser tests
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(firefox39 fixed)
RESOLVED
FIXED
mozilla39
Tracking | Status | |
---|---|---|
firefox39 | --- | fixed |
People
(Reporter: mccr8, Assigned: mccr8)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
Bug 1073352 made us do twice as many GCs and CCs at the end of tests in the parent process. Ryan noticed that we're hitting more timeouts in the browser chrome tests, which may be related. Additionally, I've investigated some e10s DOMWINDOW leaks in the parent process, and they are just showing up because we asynchronously do the collection stuff, before the child finishes, but we really need to clean up the child.
This slightly cheesy patch makes ShutdownLeaksCollector not do anything in the parent process. Instead, we print out the "Completed ShutdownLeaks collections in process" message right before we print out "TEST-START | Shutdown" to match the current behavior.
This doesn't really seem to improve any of the overall Linux32 times, but maybe it will help with individual tests that create a bunch of windows, and thus have a ton of junk at the end.
Assignee | ||
Comment 1•10 years ago
|
||
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8572056 [details] [diff] [review]
Don't run extra collections in the parent process for the mochitest window leak detector.
Oops, I forgot to flag this for review.
Attachment #8572056 -
Flags: review?(jmaher)
Comment 3•10 years ago
|
||
Comment on attachment 8572056 [details] [diff] [review]
Don't run extra collections in the parent process for the mochitest window leak detector.
Review of attachment 8572056 [details] [diff] [review]:
-----------------------------------------------------------------
thanks!
Attachment #8572056 -
Flags: review?(jmaher) → review+
Assignee | ||
Updated•10 years ago
|
Attachment #8572857 -
Flags: review?(jmaher)
Assignee | ||
Updated•10 years ago
|
Attachment #8572857 -
Attachment is obsolete: true
Assignee | ||
Comment 5•10 years ago
|
||
Comment 6•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in
before you can comment on or make changes to this bug.
Description
•