Closed Bug 1052224 Opened 5 years ago Closed 5 years ago

Disable leak checking in content processes

Categories

(Testing :: Mochitest, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla34

People

(Reporter: billm, Assigned: mccr8)

References

Details

Attachments

(1 file, 1 obsolete file)

On desktop, the content process pretty much always crashes before it gets a chance to report leaks. If we fix the crash, then leaks appear. It seems most reasonable to disable content process leak checking, fix the crashes, and eventually fix the leaks and re-enable.

Kyle, what effect will that have on your b2g work?
Flags: needinfo?(khuey)
None, yet.  Go ahead and do this.
Flags: needinfo?(khuey)
See Also: → 1051230
Assignee: nobody → continuation
Blocks: 1035454
Component: DOM: Content Processes → Mochitest
OS: Linux → All
Product: Core → Testing
Hardware: x86_64 → All
Comment on attachment 8477089 [details] [diff] [review]
Disable leak checking in content processes.

try run: https://tbpl.mozilla.org/?tree=Try&rev=0ff144b4e0d8

I'm pretty sure the M3 orange is from the other patch in that stack, as it is a shutdown assertion.
Attachment #8477089 - Flags: review?(jmaher)
Comment on attachment 8477089 [details] [diff] [review]
Disable leak checking in content processes.

Review of attachment 8477089 [details] [diff] [review]:
-----------------------------------------------------------------

::: build/automationutils.py
@@ +353,5 @@
>      return
>  
>    # totalBytesLeaked was seen and is non-zero.
> +  # For now, ignore tab process leaks. See bug 1051230.
> +  if (totalBytesLeaked > leakThreshold) and (not processType or processType != "tab"):

so we will still get a warning printed.  Can we somehow indicate in the log file in the else condition below that we are ignoring this as it is a tab process?
Attachment #8477089 - Flags: review?(jmaher) → review-
Yeah, that's probably a good idea.
Attachment #8477089 - Attachment is obsolete: true
Attachment #8477518 - Flags: review?(jmaher)
Comment on attachment 8477518 [details] [diff] [review]
Disable leak checking in content processes.

Review of attachment 8477518 [details] [diff] [review]:
-----------------------------------------------------------------

thanks, that looks much better!
Attachment #8477518 - Flags: review?(jmaher) → review+
https://hg.mozilla.org/mozilla-central/rev/f8ac66a28f2f
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.