leakcheck | default process: 4 bytes leaked (nsTArray_base) with wayland enabled (but not used)

RESOLVED FIXED in Firefox 65

Status

()

defect
RESOLVED FIXED
8 months ago
4 months ago

People

(Reporter: glandium, Assigned: stransky)

Tracking

unspecified
mozilla65
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox65 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Description

8 months ago
See https://hg.mozilla.org/try/rev/b84b5c8afaf7046fc5f969f9a91c29e09795c0a2

e.g.

[task 2018-11-07T03:46:12.974Z] 03:46:12     INFO - == BloatView: ALL (cumulative) LEAK AND BLOAT STATISTICS, default process 1056
[task 2018-11-07T03:46:12.975Z] 03:46:12     INFO - 
[task 2018-11-07T03:46:12.976Z] 03:46:12     INFO -      |<----------------Class--------------->|<-----Bytes------>|<----Objects---->|
[task 2018-11-07T03:46:12.977Z] 03:46:12     INFO -      |                                      | Per-Inst   Leaked|   Total      Rem|
[task 2018-11-07T03:46:12.978Z] 03:46:12     INFO -    0 |TOTAL                                 |       42        8| 3733887        1|
[task 2018-11-07T03:46:12.987Z] 03:46:12     INFO - 1764 |nsTArray_base                         |        8        8| 1700362        1|
[task 2018-11-07T03:46:12.988Z] 03:46:12     INFO - 
[task 2018-11-07T03:46:12.989Z] 03:46:12     INFO - nsTraceRefcnt::DumpStatistics: 1898 entries
[task 2018-11-07T03:46:12.990Z] 03:46:12     INFO - TEST-INFO | leakcheck | default process: leaked 1 nsTArray_base
[task 2018-11-07T03:46:12.995Z] 03:46:12    ERROR - TEST-UNEXPECTED-FAIL | leakcheck | default process: 8 bytes leaked (nsTArray_base)

This happens on all mochitests (and some others) jobs on debug builds, with wayland enabled, but running on X11.
Assignee

Comment 2

8 months ago
Thanks, I'll look at it.
Assignee: nobody → stransky
Reporter

Comment 3

8 months ago
Can you take bug 1504904 too?
Assignee

Comment 4

8 months ago
(In reply to Mike Hommey [:glandium] from comment #3)
> Can you take bug 1504904 too?

Sure.
Assignee

Comment 5

8 months ago
As we have only two wl_display connections it's a bit overkill to track it at nsCOMPtr array
and rather use a fixed-length one.
Attachment #9023755 - Attachment is obsolete: true
Assignee

Updated

8 months ago
Keywords: checkin-needed

Comment 7

8 months ago
Pushed by ebalazs@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2f1158e5e0ce
Don't use nsCOMPtr array to track Wayland display connections, r=jhorak
Keywords: checkin-needed

Comment 8

8 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/2f1158e5e0ce
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Depends on: 1527556
You need to log in before you can comment on or make changes to this bug.